web-dev-qa-db-fra.com

double libre ou corruption (fasttop)

La section suivante de mon code me donne ce message lors de l'exécution * glibc détecté ./ a.out: double libre ou corruption (fasttop): 0x08e065d0 * *

j'ai parcouru le code plusieurs fois mais je ne peux pas voir comment je fais un mauvais usage de la free (temp2)

bool found= false;
int x=0;
for ( x=0; x<=312500; x++)
{
    while (count <=32)
    {
        fscanf (file, "%d", &temp->num);  

        temp->ptr=NULL;

        newNode = (NODE *)malloc(sizeof(NODE));
        newNode->num=temp->num;
        newNode->ptr=NULL;

        if (first != NULL)
        {
            temp2=(NODE *)malloc(sizeof(NODE));

            temp2=first;
            while (temp2 != NULL && !found)
            {
                if (temp2->num == newNode->num) 
                {found=true;}

                temp2= temp2->ptr;
            }

            free(temp2);

            if (!found)
            { 
                last->ptr=newNode;
                last=newNode;
                count=count+1;
            }   
        }   
        else  
        {
            first = newNode;
            last = newNode;
            count=count+1;
        }

        fflush(stdin);
    }
14
Thair Abdalla

Le problème est ici:

        temp2=first;

Fondamentalement, lorsque vous libérez temp2, vous libérez d'abord, pas la mémoire allouée ici:

        temp2=(NODE *)malloc(sizeof(NODE));

, qui reste une fuite de mémoire, car après l'affectation, il ne peut plus être libéré.

De plus, votre code a probablement plus de problèmes (l'un est que vous ne devriez pas utiliser fflush sur un flux d'entrée), mais sans plus de détails, c'est impossible à dire.

16
Paul92