I need to know if I am using getline correctly

Question

I am using the getline() function in C and it keeps giving me seg faults when I use it more that once, as in for an array. Here is how I've written it:

temp = (char *)malloc(sizeof(char)*clen); 
read = getline(&temp, &clen, stdin);
tn = strtok(temp, ",");
strcpy(travel[tripnum].name, tn);
tn = strtok(NULL, ",");
travel[tripnum].country = tn;
free((void *) temp);

Please let me know if I am declaring something incorrectly

Answer 1

As seen in this getline tutorial you need to allocate (clen + 1). One extra for the terminal NULL.

Answer 2

Did you try doing this temp = (char *)malloc(sizeof(char)*clen+1);

Because of a null terminated string

Answer 3

Try using this along with what others have told. I feel in getline function clen should be used without the ampersand. Like read = getline(&temp, clen, stdin);

Answer 4

Your tn variable (result of strtok() ) points inside the temp buffer.

The temp buffer is destroyed in the last line of your snippet, however one of the tn pointers (to the inside of temp ) has been saved in travel[tripnum].country .

This travel[tripnum].country is a dangling pointer and all accesses through it are invalid.