LinkedList implementation

[rire1979]

I'm preparing for interviews and I want to get comfortable with C after years of development in Java and C#. I decided to start with the implementation of basic data structures and this brings me to the LinkedList.

Program is:

struct Node {
    struct Node* next;
    int data;
};

struct LinkedList {
    struct Node* head;
    struct Node* tail;
};

int main(int argc, char** argv) {

    struct LinkedList *list = NULL;
    printf("Declared the list");
    LLInitialize(list);
    printf("Initialized the list");
}

void LLInitialize(struct LinkedList *list) {

    list = (struct LinkedList*)malloc(sizeof(struct LinkedList));
    list->head = NULL;
    list->tail = NULL;
}

After the call to the LLInitialize() in the debugger I do not see the head/tail fields being NULL. They just are not there as if the malloc was never called. Why is that?

[Memloop]

That's because the list parameter you're using to save the return value of malloc, is a local variable. The list variable you have declared outside the LLInitialize will still point to NULL even after you've called LLInitialize. If you want to make the changes appear outside LLInitialize (and thus avoid a memory leak), the parameter should be a struct LinkedList **list instead (you also need to edit the function to reflect that).