Thread: Cannot find ( before ; error

EDIT: I have fixed most of the errors in my code. Here is the list of errors:

myMalloc.c: In function âmyMallocâ:
myMalloc.c:45: error: expected â)â before â;â token
myMalloc.c:60: error: expected â)â before â;â token
myMalloc.c:71: warning: assignment from incompatible pointer type
myMalloc.c:78: warning: initialization from incompatible pointer type
myMalloc.c: In function âfreeâ:
myMalloc.c:94: warning: dereferencing âvoid *â pointer
myMalloc.c:94: error: invalid use of void expression
myMalloc.c:99: error: expected â)â before â;â token
myMalloc.c:109: error: expected â)â before â;â token

Here is the entire code:

Code:

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

#define USED 1;
#define FREE 0;
#define MEM_SIZE 1024;

typedef struct header header;

struct header {
    unsigned int size :31;            //Number of units of block.
    unsigned int status :1;            //Whether block is free or used.
    header    *prev,                    //Pointer to the next header.
            *next;                    //Pointer to the previous head.
};

void *myMalloc(unsigned int);
void free(void*);

header *base_ptr;                    //Pointer to the base header.

int main(int argc, char* argv[])    //Driver.
{
    char *p1, *p2;
    
    p1 = myMalloc(45);
    p2 = myMalloc(32);
    
    printf("%p\n", p1);
    
    return 0;
}

void *myMalloc(unsigned int num_bytes)
{
    header *p;                        //Pointers to current and previous headers.
    header temp;

    if(base_ptr == NULL)            //If myMalloc has not been called in program yet.
    {   /***********ERROR LINE BELOW**************/
        if((base_ptr = (header*)malloc(MEM_SIZE+(MEM_SIZE%sizeof(header))+sizeof(header))) == NULL) //MEM_SIZE header units.
        {
            printf("Malloc fail\n");//Malloc failed to allocated requested memory.
            return NULL;
        }
        base_ptr[0].size = MEM_SIZE;        //Sets the initial block equal to the total size set by MEM_SIZE.
        base_ptr[0].status = FREE;            //Declares the initial block as free memory.
        base_ptr[0].next = NULL;            //No other block so the first block's ptr field is NULL.
        base_ptr[0].prev = NULL;
    }
    
    p = base_ptr;                    //Set's the current header pointer to the first header.
    
    while(p != NULL)                //While the point has not reach the end of the list.
    {
        if((p->status == FREE) && (p->size >= num_bytes))        //If the current field is free.
        {
            temp.size = p->size;
            temp.prev = p->prev;
            
            p->size = num_bytes;    //Size of payload.
            p->status = USED;        //Memory block is now allocated.
            /*If the requested size is exactly equal to the free block, pointers stays the same.*/
            
            if(temp.size > num_bytes)
            {
                p->next = (char*)p + (num_bytes*sizeof(header));    //Cast to char pointer to handle pointer arithmetic.
                p->next->size = temp.size - num_bytes;            //Cause errors at end of memory?
                p->next->status = FREE;
                p->next->prev = p;
                p->next->next = temp.next;
            }
            
            char *c = p;
            return c + sizeof(header);
        }

        p = p->next;
    }
    
    printf("myMalloc cannot allocate the requested amount of memory\n");
    return NULL;
}

void free(void *p)
{
    char *c;                //Used to simplify pointer arithmetic.
    c -= sizeof(header);
    p = c;
    header h = (header)*p;
    h.status = FREE;
    
    if(h.prev != NULL)
    {
        if(h.prev->status == FREE)
        {
            h.prev->size += h.size + sizeof(header);
            h.prev->next = h.next;            
            p = h.prev;
        } 
    }
    
    if(h.next != NULL)
    {
        if((h.next->status) == FREE)
        {
            h.size += h.next->size + sizeof(header);
            h.next = h.next->next;
        }
    }
}

EDIT2: As you can see I also have very little experience dealing with void pointers.

Here's a hint:

Code:

#define USED 1;
#define FREE 0;
#define MEM_SIZE 1024;