Thread: binary files

Code:

    for(i = 0; i<= length; i++)
    {
		file[i] = *token[2];
		*token[2]++;
    }

In these sorts of situations, it's better to use strcpy() instead of copying the string character by character yourself. This will help you to avoid making mistakes like in the above code. This is what you meant to do:

Code:

    char file[length+1];
    for(i = 0; i<= length; i++)
    {
		file[i] = token[2][i];
    }

token is not a 2D array

token is an array of what I assume are strings. You can index into a string using [].

But if it helps you sleep better at night, do this instead:

Code:

file[i] = token[2] + i;

i keep getting the seg. fault

Are you sure the data that is getting passed to the function is valid? Does token[2] point to a NULL terminated string?

The input is tokenized with strtok() with the space delimeter
token [2] is the 3rd token in the input.
so it does not have the '\0' im guessing.

does it need it?

Are you sure you used strtok() correctly?

And to answer your question: yes, token[2] needs to be null terminated. If it isn't then

Code:

int length = strlen(token[2]);

will give you a seg fault.

check your pm for full program

ah success!
forget the \n as a delimeter in my other function