Thread: Dynamic string input

So I'm trying to create a dynamically allocated user input string. I thought I had everything figured out but theres seems to be a problem when my program reaches 16 char's. Any string smaller than 16 chars seems to work completely fine, and will print out on the screen. Once the string length goes beyond that printf displays garbage. I sat there for about 2 hours trying to figure out why but I can't seem to get it.

I even use printf(%s) RIGHT before the function exits and the string prints out completely fine.

Also, am I even going about this the right way?

Code:

#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
#include <math.h>
#include "lab4.h"

int main()
{
	char *dynamic_string;
	
	dynamic_string = (char *)malloc(sizeof(char));
	if(dynamic_string == NULL)
	{
		printf("Error");
		return 1;
	}

	user_input(dynamic_string);

	printf("%s\n", dynamic_string);
	return 0;
}

Code:

#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
#include <math.h>

void user_input(char *input_array)
{
	int i;
	int j;
	int flag = 0;
	int string_size = 0;
	int num_allocated = 1;
	char * temp;	

	for(i=0; flag!=1; i++)
	{
		scanf("%c", &input_array[i]);
		string_size++;

		if(string_size == num_allocated)
		{
			temp = (char *)malloc(num_allocated*2);
			if(temp == NULL)
			{
				printf("Error");
				flag = 1;
			}
	
			for(j=0; j<=string_size; j++)
			{
				temp[j] = input_array[j];
			}

			free(input_array);
			input_array = (char *)malloc(num_allocated*2);	
			if(input_array == NULL)
			{
				printf("Error");
				flag = 1;
			}

			for(j=0; j<=string_size; j++)
			{
				input_array[j] = temp[j];
			}
			
			free(temp);
			temp = NULL;
			num_allocated *= 2;
		}

		if(input_array[i] == '\n')
		{
			input_array[i] = '\0';
			printf("TEST = %s\n", input_array);
			flag = 1;
		}
	}
}

Ah I thought it would be something along those lines, I originally did all the allocating inside of main and only had the user_input function hold the scanf() code to input the string, but then decided to change it because I didn't know if that was the proper way to code. I'm redoing one of my old labs to use dynamic memory, so I don't know where I should be doing the allocating, inside the functions or inside of main?

Part A

In this file, create a new C function which can read in whole lines of input from the user of up to 126 characters in length, and store them in a character string passed-in as a function argument. Lines will be separated (only) by newline characters. Note that empty lines are valid entries as well. All other characters should be read and added into the string.

Create another new C file, as before, and in it write a new function that will use the previous function to read in up to 128 lines from the user until they enter the special line of a "." by itself, or until they enter a 128th line. These should be read into a 128x128 two-dimensional array passed in as a function argument.
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Also, I'm an amateur when it comes to programming, so why can't you modify where a an argument passed pointer points to inside of a function?

So heres my failed attempt at trying to fix this horrid code using char **'s.

Code:

int main()
{
	char *dynamic_string;
	char **string_ptr;
	int i=0;
	
	dynamic_string = (char *)malloc(sizeof(char));
	if(dynamic_string == NULL)
	{
		printf("Error");
		return 1;
	}

	string_ptr = &dynamic_string;
	user_input(string_ptr);
	printf("%s\n", dynamic_string);

	return 0;
}

Code:

void user_input(char **input_array)
{
	int i;
	int j;
	int flag = 0;
	int string_size = 0;
	int num_allocated = 1;
	char * temp;	

	for(i=0; flag!=1; i++)
	{
		scanf("%c", *(&input_array[i]));
		string_size++;

		if(*input_array[i] == '\n')
		{
			*input_array[i] = '\0';
			flag = 1;
		}
		else
		{
			if(string_size == num_allocated)
			{
				temp = (char *)malloc(num_allocated);
				for(j=0; j<string_size; j++)
				{
					temp[j] = *input_array[j];
				}

				free(*input_array);
				*input_array = (char *)malloc(num_allocated*2);
			
				for(j=0; j<string_size; j++)
				{
					*input_array[j] = temp[j]; 
				}
				free(temp);
				temp = NULL;
				num_allocated *= 2;
			}
		}
	}
}

This is my assumption of how its supposed to work. String_ptr points to dynamic_string, which is the memory I would like to allocate. I pass string_ptr(which points to the address of dynamic_string) to user_input where I can manipulate and dynamically allocate dynamic_string.

So inside user_input.
*input_array is technically the address of dynamic_string?
and *input_array[index] allows me to input a char into dynamic_string?

I think I have a fatal misunderstanding of pointers, or maybe its just amateur hour.