Question

This code causes a Segmentation Fault:

int main(){

    char *p;
    char a[50] = "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
    p = (char *)malloc(50*sizeof(char));

    if(!p){
            cout << "Allocation Failure";
            cout << "\n";
    }
    else{
            cout << "Allocation Success";
            cout << "\n";
            p = a;
            cout << p;
            cout << "\n";
            free(p);
    }

    return 0;
}

The output after executing this program is:

Allocation Success

aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

Segmentation fault

I am not able to find the bug. What may be reason?

Answer 1

You’re calling free on a block of memory that wasn’t allocated using one of the malloc methods.

When you do: p = a you’re assigning to p the memory used by the stack array a. That memory wasn’t allocated using malloc and hence it can’t be freed using free.

Furthermore with that re-assignment, you’ll lose track of the block that you originally allocated with malloc and assigned to p, causing a memory leak.