Stack Linked List query

[Josh Heathcote]

Would you say that my notes on the code are correct and if not will you change them so that they are, thank you.

void Push(int x)
{
	// Create a space in the heap to store data.
	struct Node* temp = (struct Node*)malloc(sizeof(struct Node*)); 


	// here we are saying that temp.data = x
	temp->data = x;


	// Here we are saying that the temp.link is pointing to top.link
	temp->link = top;


	// Here we are saying that top is pointing to temp.
	top = temp;
}

[Salem]

> struct Node* temp = (struct Node*)malloc(sizeof(struct Node*));
Except you're allocating the wrong amount of space.

Try writing malloc calls like this (without the cast).
struct Node* temp = malloc(sizeof(*temp));

Tell us what warnings you get from malloc, and we'll tell you the correct way to fix it.