Динамические структуры ошибка

[Praud]

#include "stdafx.h"
#include "stdio.h"
#include "conio.h"
#include "malloc.h"

typedef struct listnode
{
	int data;
	struct listnode *next;
}LIST;

LIST *create_list();
LIST *head;
void display(LIST *st);


int main(LIST *lst, LIST *head, LIST *current, LIST *prev)
{
	create_list();
	printf("Current list:\n");
	display(head);
	while(lst)
	{
	if(lst->data<=0)
	{
	lst=(LIST*)malloc(sizeof(LIST*));
	lst->data=222;
	prev->next=lst;
	prev=lst;
	}
	}
	printf("Modified list:\n");
	display(head);
	_getch();
	return 0;
}

LIST *create_list()
{
	int item;
	LIST *current, *prev;
	char c;

	head=current=prev=(LIST*)malloc(sizeof(LIST*));
	printf("Item :\n");
	scanf_s("%d", &item);
	current->data=item;
	do{
		current=(LIST*)malloc(sizeof(LIST*));
		printf("Item: \n");
		scanf_s("%d", &item);
		current->data=item;
		prev->next=current;
		prev=current;
		printf("end? Y/N?\n");
		c=_getch();
	}
	while(c!='y');
	current->next=NULL;
	return head;
}

void display(LIST *lst)
{
	LIST *cur=lst;
	while(cur)
	{
		printf("%d-->", cur->data);
		cur=cur->next;
}
	printf("NULL\n");

}

Добавить в список L после каждого отрицательного элемента значение Х.

Само задание расположено в функции main. Там и ошибка.
Что я не так написал?

А, да вместо X я использовал 222

[kidiam]

Главная ошибка в цикле, который добавляет твой 222.

Во-первых, там условие "<=" вместо "=", что конечно не критично.
Во-вторых, главная функция как-то странно определена. Вместо привычных параметров argv и argc (которые обычно вообще пропускают) определяется множество параметров. Лучше определить так

int main()
{
     LIST *current, *lst;

В-третьих, список head определен дважды: локально в main и глобально. Лучше делать все определения локальными.
В-четвертых, созданные элементы (выделенные) функцией malloc не удаляются (не освобождаются) функцией free. Например так

void delete_list(LIST *p)
{
    LIST *cur;
    for(;p;) {
        cur = p;
        p = p->next;
        free(cur);
    }
}

Последнее, и самое главное, сам цикл лучше сделать так

    for(current=head; current; current=current->next)
    {
        if(current->data<0)
        {
            lst = (LIST*)malloc(sizeof(LIST*));
            lst->data = 222;
            lst->next = current->next;
            current->next = lst;
            current = lst;
        }
    }

Т.е. сначала выделяем память под новый элемент lst, затем заполняем все его поля.
Для текущего (current) элемента изменяем ссылку на следующий элемент - новое значение и есть, только что созданный, элемент lst.
Последняя строка изменяет значение текущего элемента на следующий, только что вставленный lst. Если это не сделать, то главный цикл (for) перейдет на следующий элемент после текущего, но мы там добавили наш новый элемент, т.е. следующим в цикле будет проверен только что вставленный lst элемент, а если он тоже окажется отрицательным - получим грустную ситуацию.