1

Alguém poderia me ajudar com o seguinte código?

void removeRepetidos(Lista *l)
{

    Elemento *aux = (*l);
    Elemento *aux2;
    Elemento *aux3;

    while(aux!=NULL)
    {
        aux2 = aux->prox;

        while(aux2 != NULL)
        {
            if(aux->num == aux2->num)
            {

                aux2->ant->prox = aux2->prox;

                if(aux2->prox!=NULL)
                {
                    aux2->prox->ant = aux2->ant;
                }


                aux3 = aux2;

            }


            aux2 = aux2->prox;
            free(aux3);
        }

        aux = aux->prox;
    }
}

Ele está crashando ao executar, ao retirar a função de desalocação free() o programa funciona perfeitamente. Nâo encontrei nenhum erro na lógica do meu código, fiz vários testes de mesa.

1 Resposta 1

2

O ponteiro aux3 está sendo deletado duplamente,

void removeRepetidos(Lista *l)
{
  Elemento *aux = (*l);
  Elemento *aux2;
  Elemento *aux3;

  while (aux!=NULL)
  {
    aux2 = aux->prox;
    while (aux2 != NULL)
    {
      if (aux->num == aux2->num)
      {
        aux2->ant->prox = aux2->prox;
        if (aux2->prox != NULL)
        {
          aux2->prox->ant = aux2->ant;
        }

        aux3 = aux2;
        aux2 = aux2->prox; // <---
        free(aux3);        // <--- incluir aqui
        continue;          // <---
      }

      aux2 = aux2->prox;
      // free(aux3); // <--- tirar daqui
    }

    aux = aux->prox;
  }

}

2
  • 1
    Mas se eu liberar o aux3 antes do incremento aux2 = aux2 -> prox não terá problema? mesmo que o ponteiro aux3 receba o de aux2? 29/10/2018 às 2:37
  • veja a nova edição
    – zentrunix
    29/10/2018 às 2:43

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .