Free() Funktion verursacht Programm zum Absturz bringt

Question

ich derzeit zugewiesenen Speicher zu versuchen, aber durch so tut es das Programm zum Absturz verursacht. Ich bin neu in C und Programmierung im Allgemeinen, und wäre großartig, um Hilfe mit dem Problem und allen weiteren Problemen zu erhalten, die aus meiner Unerfahrenheit stammen können.

Pool* allocatePool(int x); 
void freePool(Pool* pool); 
void store(Pool* pool, int offset, int size, void *object); 

typedef struct _POOL 
{ 
    int size; 
    void* memory; 
} Pool; 

int main() 
{ 
    printf("enter the number of bytes you want to allocate//>\n"); 
    int x; 
    Pool* p; 
    scanf("%d", &x); 
    p=allocatePool(x); 
    freePool(p); 
    return 0; 
} 

/* Allocate a memory pool of size n bytes from system memory (i.e., via malloc()) and return a pointer to the filled data Pool structure */ 

Pool* allocatePool(int x) 
{ 
    static Pool p; 
    p.size = x; 

    p.memory = malloc(x); 
    printf("%p\n", &p); 

    return &p;//return the address of the Pool 
} 

/* Free a memory pool allocated through allocatePool(int) */ 
void freePool(Pool* pool) 
{ 
    free(pool); 
    printf("%p\n", &pool); 
} 

Answer 1

Sie müssen denselben Speicher freigeben, den Sie zugewiesen haben. hier:

p.memory = malloc(x); 

Sie zuteilen x Bytes p.memory. Das heißt, hier:

free(pool); 
printf("%p\n", &pool); 

sollten Sie den gleichen Speicher freigeben. Sie versuchen, das Pool-Objekt stattdessen free, aber das ist nicht auf dem Heap zugeordnet. In Ihrer Implementierung ist es ein statisches Objekt, das den einzelnen Pool enthält. Versuchen, free Speicher, der nicht auf dem Heap zugeordnet wurde, ist undefiniertes Verhalten, in Ihrem Fall ein Absturz.

Der obige Code zeigt auch ein weiteres Missverständnis über free: Es ist nicht der Griff an den zugewiesenen Speicher nicht ändert. Es markiert nur die zuvor zugewiesenen Bytes als wieder verwendbar. Ihr Programm muss sich gegen den Zugriff auf den Speicher durch den Zeiger schützen, dessen Speicher Sie haben free d.

Außerdem ist &pool nicht die Adresse des Pool-Objekts, sondern die Adresse der lokalen Zeigervariablen pool.

den Absturz zu beheben, um Ihre Funktion ändern:

void freePool(Pool *pool) 
{ 
    if (pool) free(pool->memory); 
}