Linked list problem

Hello, I am having this problem with a linked list. In a program I have created a method that links nodes together except when I check each node's data, all of them contain the same thing. This means that the lists is being created but not 100% correct. here is some of the code.
The data that is being looked is a created struct called String that is defined as :

Code:

typedef struct {
       char *str; /* str is a dynamic array of characters*/
       int len;  /* number of characters */
    } String;

here is the method that creates the linked list:

Code:

void addNode(listNode** headRef, String data) {   
  listNode* newNode = (listNode *)malloc(sizeof(listNode));
  newNode->data = data;
  newNode->next = *headRef;
  *headRef = newNode; 
}

Here is the main method:

Code:

String data;// data recieved from user
 listNode* list; // operation will be performed on this
 listNode* printer;// used to traverse through list

int main(void) {
   
   data.str = (char*)malloc(sizeof(char)); //allocate memory
   printf("Enter a string:\n");
   scanf("%s", data);
   ListInitialize(list); //intitalize list
   addNode(&list, data); // add a node to the list
   printf("Enter another string:\n");
   scanf(" %s", data.str);
   addNode(&list,data);//add another node to the list
   printf("%d\n",Length(list));// check the length of list
   
   /** print each string in the list */
   for(printer = list; printer != NULL; printer = printer->next) {
       printf("%s\n", printer->data);
   }   
   freeList(list); 
   return 0;
}

When I run the code it looks like this:

Enter a string:
name1
Enter another string:
name2
2
name2
name2

the struct data contains a str ,which is a pointer which is initialized to (char*)malloc(sizeof(char));
u have to remember that str is a pointer,and so this doesnt contain the actual value,rather just the address of where a string array is to be found.
so when u use scanf once, str is pointing to name1;
and when u use scanf for another time, str is pointing to name 2;


u have to remember its value has been updated to point to name2.
so both link1.data and link2.data contain the address of str which in turn points to name2.

yeah after looking at my code I realized my problem. Like you said it references. I just forgot to use the strcpy() function which fixes the problem thanks

Code:

sizeof(char)

is always 1. It's guaranteed to always be 1.

Code:

scanf("%s", data);

You might want to look into fgets() or other ways of getting input into a string. See the FAQ. [edit]Get a line of text from the user/keyboard (C)[/edit]

And you shoudn't cast malloc(); see the FAQ again. [edit]Casting malloc[/edit]

> data.str = (char*)malloc(sizeof(char));
1. This only allocates a single char - which is actually useless for C-Strings which always end with a \0 (so you've no room for any actual data).
2. Read the FAQ on why casting malloc is bad in C.

> scanf("%s", data);
Does your compiler even complain at this line?
Or are you just ignoring all warnings?
In any event, you just trashed something you shouldn't have, so the rest of the code becomes a moot point.