Memory error when I try to get list data from a text file in C

[Avon]

Hello there.

I have a problem with text files in C. Basically what I do is, let the user save the data from a singly-linked list, in to a .txt using this code:

while (list!=NULL)
    {
    fprintf(fp,"%s %s %s %c %f %d\n",list->kodikos,list->perigrafi,list->etaireia,list->diathesimotita,list->timi,list->posotita);
      list=list->next;
    }

Then in order to be sure, I check the .txt file so as to see that all the data is stored there. And indeed it is.

However when I try to load the list on my program, I get a memory (?) error. Here's the code I use:

    while (!feof(fp))
        {
          fscanf(fp,"%s %s %s %c %f %d\n",buffer_kod,buffer_per,buffer_et,&temp->diathesimotita,&temp->timi,&temp->posotita);
          temp->kodikos=strdup(buffer_kod);
          temp->perigrafi=strdup(buffer_per);
          temp->etaireia=strdup(buffer_et);
          printf("This is the current string: %s\n", temp->kodikos);
            proion=insert(proion,temp);
          }
    fclose(fp);
    printf("File '%s', loaded.\n", s);
    menu();

"proion" is the pointer to the full list, while "temp" is the pointer to new node.
Function "insert" is a typical function that gets new nodes and puts them sorted on the full list:

struct lista *insert (struct lista *p,struct lista *list)
{
    int cond;
    struct lista *n;
 
    if (p==NULL)
    {
        p=(struct lista *)malloc(sizeof(struct lista));
        p->kodikos=strdup(list->kodikos);
        p->perigrafi=strdup(list->perigrafi);
        p->etaireia=strdup(list->etaireia);
        p->timi=list->timi;
        p->diathesimotita=list->diathesimotita;
        p->posotita=list->posotita;
        p->next=NULL;
        printf("Product not found. Added to the full list!\n\n");
    }
    else if ((cond = strcmp(list->kodikos,p->kodikos)) == 0)
    {
        p->posotita = p->posotita + list->posotita;
          printf("Product found. Quantity updated!\n\n");
    }
    else if (cond < 0)
    {
         n=(struct lista *)malloc(sizeof(struct lista));
        n->kodikos=strdup(list->kodikos);
        n->perigrafi=strdup(list->perigrafi);
        n->etaireia=strdup(list->etaireia);
        n->timi=list->timi;
        n->diathesimotita=list->diathesimotita;
        n->posotita=list->posotita;
        n->next=p;
        p=n;
        printf("Product not found. Added to the full list!\n\n");
    }
    else if (cond > 0)
    {
        p->next = insert(p->next, list);
    }
 
    return p;
 
 
}

The program manages to end the (!feof) loop succesfully, since printf on line 7 keeps producing the right results and the insert function gives me the correct messages of "product not found. Added to the list" etc. When the loop ends I get the printf message on line 11 that says "File loaded" and the menu comes back again. However when it does come up again I'm not allowed to type anything and I get a memory error. Anyone know why's that happening?

Thanks. ;)

[Savage]

The program manages to end the (!feof) loop succesfully, since printf on line 7 keeps producing the right results and the insert function gives me the correct messages of "product not found. Added to the list" etc. When the loop ends I get the printf message on line 11 that says "File loaded" and the menu comes back again. However when it does come up again I'm not allowed to type anything and I get a memory error. Anyone know why's that happening?

Thanks. ;)

Have u allocated memory for temp?

Savage

[Avon]

No... Followed your suggestion and it seems to be working, although... care to explain why? I mean if it was a memory problem for "temp" why does it allow me to enter values on the first place? :S

Thanks a lot for finding the problem. :)

[Savage]

No... Followed your suggestion and it seems to be working, although... care to explain why? I mean if it was a memory problem for "temp" why does it allow me to enter values on the first place? :S

Thanks a lot for finding the problem. :)

U allways need to allocate memory for all pointers to struct.

When memory is not allocated u can input data, but that data is garbage.

Compiler wont send any warning,it's analog with call to scanf without adress operator.

Savage

[Savage]

Thanks a lot for finding the problem. :)

I'm more than happy!

Savage