[help] - adding to end of linkedlist

Hi,

I am having problems with the following code, it seems to have gone into an infinite loop while trying to add current to the end of the list. I would much appreciate someone explaining where I have gone wrong and how I should fix it.

Thanks heaps

Code:

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

struct list_el {
   char *val;
   struct list_el * next;
};

typedef struct list_el item;

int main() {
	item *current, *head;
	head = NULL;
	current = NULL;

	// create file
	FILE *battingFile;

	// open file
	battingFile = fopen("BATTING.txt","r");

	// error if cannot open
	if (battingFile == NULL) 
	{
		printf("Can't open input file!\n");
	}

	char line[300];

   	while (fgets(line, 300 ,battingFile)) 
	{	
		while(line[0] != '#')
		{	
			//printf("%s", line);
			current = (item *)malloc(sizeof(item));
			if(head == NULL)
				{	
					// add info to front of list
					head = (item *)malloc(sizeof(item));
					head->val = line;
					current = head;
				}
				else
				{
					// go to the end of list
					while(current->next != NULL)
					{
						current = current->next;
					}
					
					// add to end of list
					current->val = line;
				}
		}
  	}
return 1;
}

So, if it's a linked list, maybe you should be ... setting links? Right now you have 300 disconnected nodes, so you can't walk through the list at all.

I modified the code to the following to actually connect the list, but am still the program is hanging, I know something is wrong but I just can't see it.

Code:

while (fgets(line, 300 ,battingFile)) 
	{	
		while(line[0] != '#')
		{	
			//printf("%s", line);
			if(head == NULL)
				{	
					// add info to front of list
					head = (item *)malloc(sizeof(item));
					head->val = line;
					head->next = NULL;
				}
				else
				{
					// go to the end of list
					while(current == NULL)
					{	
						current = (item *)malloc(sizeof(item));
						current->val = line;
						current->next = NULL;
						current = current->next;
					}
				}
		}
  	}

Nope, still not building a list there,
You'll need to set something->next equal to something other than NULL for all nodes with successors.