Allocating Memory for a Structure

Im just learning C and I need help. Im trying to allocate memory for a structure call mystruct, initialize the values for the strucutre and then return a pointer to the allocated memory:

void main(){
struct mystruct pdemo;
pdemo=alloc_mystruct(buf1,buf2,val1);
}

struct mystruct *alloc_mystruct(char *name,const char *id,int val){
struct mystruct *pMyStruct;

pMyStruct->name=(char*)malloc(sizeof(name));
pMyStruct->id=(char*)malloc(sizeof(id));
pMyStruct->val=malloc(sizeof(val));

strcpy(pMyStruct->name, name);
strcpy(pMyStruct->id, id);
pMyStruct->val=val;

return pMyStruct;
}

could someone please point out what im doing wrong here?

struct mystruct
{
char *name;
char id[10];
int val;
};

and the declaration should be:
struct mystruct *pdemo;

in the main function

Greetings,

First of all: use code tags:

Code:

---

void main(){
    struct mystruct *pdemo; /* You forgot the '*', typo? */
    /* Also, at this point the compiler does not know about 'alloc_mystruct' it will assume it returns an int */
    pdemo=alloc_mystruct(buf1,buf2,val1);
}

struct mystruct *alloc_mystruct(char *name,const char *id,int val){
  struct mystruct *pMyStruct;

    pMyStruct->name=(char*)malloc(sizeof(name));
    pMyStruct->id=(char*)malloc(sizeof(id));
    pMyStruct->val=malloc(sizeof(val));

    strcpy(pMyStruct->name, name);
    strcpy(pMyStruct->id, id);
    pMyStruct->val=val;

    return pMyStruct;
}

---

Second, its 'int main', not 'void main'.
Third, provides the full code, like where's buf1, buf2, val1, MyStruct defined and how?

thanks, everything is working fine now...the function is already prototyped in the header file...the variables are being read in from console window....sorry to leave out so much...the project is rather large and i was trying to minimize the amount of code i posted in the forum...

one last question

is it necessary to use memset to initialize the structure after the memory has been allocated?