Thread: Make node code - right?

Code:

Node *MakeNode(StackE item)  
{
  Node *nptr;
  if( (nptr = malloc ( sizeof nptr) ) != NULL)     
        {
        nptr = item;     
        return item;
        }
        else
                return NULL;

} 

the structs for this are:

typedef int StackE;

typedef struct node{
StackEntry entry;
struct node *next;
}Node;

Returns: NULL if out of memory; otherwise a ptr to enough memory to hold a Node, with that node set to correct value(s), ready to be put on stack

Thanks