Erreur lors de l'exécution de free() indiquant "glibc détecté double free ou corruption"

Question

J'essaie d'exécuter un programme en utilisant un tableau que j'ai alloué manuellement avec malloc(). Je libre() ce tableau à la fin de mon programme, mais je continue à obtenir une erreur en disant *** glibc detected *** ./test: double free or corruption (!prev): 0x0000000002177010 ***

Voici mes principales:

int main(int argc, char** argv) 
{ 
    pthread_t t1, t2, t3; 
    int i = 1; 
    int k = 0; 
    Client* clients; 
    clients = (Client*) malloc((nbClients+1)*sizeof(Client)); 
    for (i = 1; i <= nbClients+1; i++) 
    { 
     printf("\n----TICKET%d----\n", i); 
     clients[i].panier = (int*) malloc(nbArticles*sizeof(int));  
     achats(clients[i].panier, &clients[i].ticket); 

     for (k = 0; k < nbArticles; k++) 
     { 
      printf("panier[%d] = %d\n", k, clients[i].panier[k]); 

     } 

     pthread_create(&t1, NULL, calcMACL, &clients[i]); 

     //calcMQUAD(clients[i].panier, &clients[i].ticket); 
     //calcMACL(clients[i].panier, &clients[i].ticket); 
     //calcMCUBE(clients[i].panier, &clients[i].ticket); 

     pthread_join(t1, NULL); 
     //free(clients[i].panier); 

    } 

    free (clients); 




    return 0; 
} 

Merci pour votre aide

Answer 1

Vous accédez en dehors des limites de la mémoire tampon que vous avez allouée, ce qui corrompt peut-être les structures de données malloc résultant de l'erreur que vous obtenez. C'est undefined behaviour.

La boucle doit être:

for (i = 0; i < nbClients+1; i++) 

Rappelez-vous, l'indice commence à 0 en C.

Answer 2

Lorsque free() échoue, il est en raison d'une corruption de mémoire dans votre programme, causée par des insectes.

Dans ce cas, c'est parce que la boucle for (i = 1; i <= nbClients+1; i++) accède au tableau hors des limites.

Answer 3

Les indices dans C commencent à partir de zéro. Le dernier index de clients est nbClients + 1 - 1 ce qui signifie que votre protège-boucle est incorrect. Je recommande d'utiliser une variable distincte pour la longueur du tableau dynamique et une macro pour l'allocation de mémoire:

#define NEW_ARRAY(ptr, n) (ptr) = malloc((n) * sizeof (ptr)[0]) 

int clientsLen = nbClients + 1; 
NEW_ARRAY(clients, clientsLen); 
for (i = 0; i < clientsLen; i++) 
    ...