【问题标题】:SIGTRAP when freeing first element of a Queue释放队列的第一个元素时的 SIGTRAP
【发布时间】:2013-05-10 00:58:22
【问题描述】:

我遇到了麻烦,每次我的函数调用“desenfileirar”时,我都有一些断点。谁能帮我?我需要打印一个二维数组,这意味着蚂蚁行进的路径。它需要从 (0,0) 开始并到达 (9,9)。我成功了,只在调试器中使用“handle SIGTRAP nostop”命令。我实现了 BFS 算法,但我没有成功将元素出列。我相信这意味着内存违规

Program received signal SIGTRAP, Trace/breakpoint trap.
In ntdll!TpWaitForAlpcCompletion () (C:\Windows\system32\ntdll.dll)
In ntdll!RtlLargeIntegerDivide () (C:\Windows\system32\ntdll.dll)
In ntdll!RtlCopyExtendedContext () (C:\Windows\system32\ntdll.dll)
#10 0x004014cd in desenfileirar (F=0x28fe74) at I:\Exercício-1\Formiga.c:60
I:\Exercício-1\Formiga.c:60:1306:beg:0x4014cd
At I:\Exercício-1\Formiga.c:60

代码如下:

#include <stdlib.h>
#include <stdio.h>
#include <stdbool.h>
#include "Fila.h"

struct st_no{
    int linha; //coordinate from array line
    int coluna; //coordinate from array column
    int plinha; //coordinate from the generator of line (father)
    int pcoluna; //coordinate from the generator of column (father)
    FILA *prox;
};

void geraFilhos(FILA **Q, FILA **gerador, short int *matriz[N][N], int *visitados[N][N]);
void print_shortest_path(FILA **F, FILA **src, FILA **dst);

/**=========================== FUNÇÕES DA FILA ===========================**/
bool vazia(FILA **F){
    return *F == NULL;
}

void criar(FILA **F){
    *F = NULL;
}

void enfileirar(FILA **F, int i, int j, int paiI, int paiJ){
    FILA *novo, *P;
    novo = (FILA *)malloc(sizeof(FILA*));
    novo->linha = i;
    novo->coluna = j;
    novo->plinha = paiI;
    novo->pcoluna = paiJ;
    novo->prox = NULL;

    if(*F == NULL)
        *F = novo;
    else{
        P = *F;
        while(P->prox != NULL)
            P = P->prox;
        P->prox = novo;
    }
}

FILA *desenfileirar(FILA **F){
    FILA *P, *ret = (FILA*)malloc(sizeof(FILA));
    if(vazia(F)){
        return NULL;
    }
    else{
        P = *F;
        ret->linha = P->linha;
        ret->coluna = P->coluna;
        ret->plinha = P->plinha;
        ret->pcoluna = P->pcoluna;
        ret->prox = NULL;
        *F = (*F)->prox;
        free(P); // HERE I HAD THE BREAKPOINTS
    }
    return ret;
}

FILA *buscar(FILA **L, int i,int j){
    FILA *P;

    P = *L;
    while(P != NULL){
        if(P->linha == i && P->coluna == j)
            return P;
        P = P->prox;
    }
    return NULL;
}

void imprimir(FILA **F){
    FILA *P;

    P = *F;
    printf("Fila:\n");
    while(P != NULL){
        printf("(%i,%i)", P->linha, P->coluna);
        printf("(%i,%i)\n\n", P->plinha, P->pcoluna);
        P = P->prox;
    }
}

FILA *atribuicao(FILA **F, int i, int j){
    FILA *aux = (FILA*)malloc(sizeof(FILA));
    aux->linha = i;
    aux->coluna = j;
    aux->prox = NULL;
    *F = aux;
    return *F;
}

/**=========================== FUNÇÕES QUE ACHAM O CAMINHO ===========================**/

void caminhar(short int *matriz[N][N], FILA *inicio,FILA *objetivo){
    FILA *abertos, *x, *fechado;
    int i, j, *visitados[N][N];

    criar(&abertos);
    criar(&fechado);

    for(i = 0; i < N; i++){
        for(j = 0; j < N; j++){
            visitados[i][j] = 0;
        }
    }

    inicio->plinha = -1;
    inicio->pcoluna = -1;
    enfileirar(&abertos,inicio->linha,inicio->coluna,inicio->plinha,inicio->pcoluna);
    while(!vazia(&abertos)){
        x = desenfileirar(&abertos);
        enfileirar(&fechado,x->linha,x->coluna,x->plinha,x->pcoluna);
        if(x->linha == objetivo->linha && x->coluna == objetivo->coluna){
            printf("Parou aqui!\n\n\n");
            break;
        }
        else{
            geraFilhos(&abertos,&x,matriz,visitados);
            visitados[x->linha][x->coluna] = 1;
        }
    }
    imprimir(&fechado);
    print_shortest_path(&fechado,&inicio,&objetivo);
}

void geraFilhos(FILA **Q, FILA **gerador, short int *matriz[N][N], int *visitado[N][N]){
    FILA *P = *gerador;

    if((P->coluna+1 < N)&&(matriz[P->linha][P->coluna+1] == 0) && (visitado[P->linha][P->coluna+1] == 0)){//direita
        P->plinha = P->linha;
        P->pcoluna = P->coluna;
        P->coluna++;
        enfileirar(Q,P->linha,P->coluna,P->plinha,P->pcoluna);
        P->coluna--;
    }
    if((P->linha+1 < N)&&(matriz[P->linha+1][P->coluna] == 0) && (visitado[P->linha+1][P->coluna] == 0)){//baixo
        P->plinha = P->linha;
        P->pcoluna = P->coluna;
        P->linha++;
        enfileirar(Q,P->linha,P->coluna,P->plinha,P->pcoluna);
        P->linha--;
    }
    if((P->coluna-1 >= 0)&&(matriz[P->linha][P->coluna-1] == 0) && (visitado[P->linha][P->coluna-1] == 0)){//esquerda
        P->plinha = P->linha;
        P->pcoluna = P->coluna;
        P->coluna--;
        enfileirar(Q,P->linha,P->coluna,P->plinha,P->pcoluna);
        P->coluna++;
    }
    if((P->linha-1 >= 0)&&(matriz[P->linha-1][P->coluna] == 0) && (visitado[P->linha-1][P->coluna] == 0)){//cima
        P->plinha = P->linha;
        P->pcoluna = P->coluna;
        P->linha--;
        enfileirar(Q,P->linha,P->coluna,P->plinha,P->pcoluna);
        P->linha++;
    }
}

void print_shortest_path(FILA **F, FILA **src, FILA **dst){
    FILA *P, *Q;
    Q = *F;
    printf("CAMINHO: \n\n\n");

    printf("(%d,%d)\n", (*dst)->linha,(*dst)->coluna);
    while((*dst)->linha != (*src)->linha && (*dst)->coluna != (*src)->coluna){
        P = buscar(&Q,(*dst)->linha,(*dst)->coluna);
        printf("(%d,%d)\n", P->plinha,P->pcoluna);
        (*dst)->linha = P->plinha;
        (*dst)->coluna = P->pcoluna;
    }
    printf("(%d,%d)\n", (*src)->linha,(*src)->coluna);
}

/**=========================== MAIN ===========================**/

#include <stdio.h>
#include <stdlib.h>
#include "Fila.h"

/*
 *
 */
 void caminhar(short int *matriz[N][N], FILA *inicio,FILA *objetivo);

int main(int argc, char** argv) {
FILE *arq = fopen("teste.txt", "r");
    int i, j;
    int tabuleiro[N][N];

    FILA *inicial, *objetivo;

    criar(&inicial);
    criar(&objetivo);

    inicial = atribuicao(&inicial,0,0);
    objetivo = atribuicao(&objetivo,N-1,N-1);

    if(!arq){
        printf("Nao deu pra ler!");
    }else{
        for(i = 0; i < N; i++){
            for(j = 0; j < N; j++){
                fscanf(arq,"%d",&tabuleiro[i][j]);
            }
        }

        printf("INICIO: (0,0)\n");
        printf("OBJETIVO: (%d,%d)\n\n", N, N);

        caminhar(tabuleiro,inicial,objetivo);
    }
    system("PAUSE");
    return (EXIT_SUCCESS);
}

/**=========================== FILA.H ===========================**/

#include <stdlib.h>
#include <stdbool.h>
#define N 10

typedef struct st_no FILA;

void criar(FILA **F);
void destruir(FILA **F);
bool vazia(FILA **F);
void enfileirar(FILA **F, int i, int j, int paiI, int paiJ);
FILA *desenfileirar(FILA **F);
void imprimir(FILA **F);
FILA *atribuicao(FILA **F, int i, int j);

【问题讨论】:

  • 据我所知,这绝对没有任何作用:if(*F == NULL){ free(*F); *F = NULL; }
  • 我很确定。我只是从我的老师那里复制的......
  • 我很乐意向他/她提及这一点。它只会增加额外的膨胀。

标签: c memory free


【解决方案1】:

拥有complete, compilable example 会有所帮助,因为您提供了名为desenfileirar 和caminhar 的函数的源代码,但您也使用了名为criar、enfileirar、vazia、geraFilhos 的函数、imprimir 和 print_shortest_path,并且您使用结构名称 FILA 而不提供定义。此外,您从不显示malloc(),而是显示对free() 的调用。

caminhar() 开头的free() 本质上是没有意义的(调用free(NULL) 保证不会做任何事情,并且仅当*f 已经是NULL 时才显式设置*f = NULL?另外,我'我相信你会同意,没有帮助),但无害。

我看到的最明显的问题是您的ret 是一个指向FILA 的指针,但您使用它时没有为其分配存储空间。这意味着当您输入desenfileirar() 函数时,会创建一个名为ret 的变量,该变量具有足够的存储空间来存储指向FILA 的指针,但没有显式分配值,然后您将其视为有效指针,并且把它写下来。 (然后你returning 它,也是......)这是未定义的行为,多次重复,幸运的是,你很幸运,这一次,它在测试期间失败了。有不止一种可能的解决方案,但没有看到你的整个程序,我不知道推荐哪一个。 (不过,最可能的解决方案是在开始使用它之前插入一行 ret=malloc(sizeof *ret);。)

**UPDATE**

既然您已经发布了附加代码,下面是我的进一步分析。这似乎是几个源文件,但它显然仍然不可编译,并且缺少 Fila.h。

在enfileirar() 中,您使用malloc() 为指向FILA 的指针分配足够的存储空间,而不是为FILA 分配足够的存储空间。在desenfileirar() 中,您在对malloc() 的调用中的第一行有语法错误。此外,如果*F==NULL 存在内存泄漏。在您需要之前,不要将malloc() 存储为ret。您在free() 行上还有一个随机的2,并且您忘记初始化ret-&gt;prox=NULL,这将在稍后的某个随机时间导致未定义的行为。在atribuicao() 中,您忘记初始化aux-&gt;prox=NULL。

您的问题几乎肯定源于您忘记初始化新 FILA 的 prox 元素的两个地方。这是因为从malloc() 返回的内存不是空白的,内容是不确定的。所以如果你不设置prox=NULL,当你遍历列表时,你会从最后走到完全随机的记忆中。

【讨论】:

  • 我已经发布了完整的代码!而且,非常感谢您的帮助!在最旧版本的代码中,我有分配 ret,但由于某种未知原因,我将其取消。
  • 现在,我已经更新了代码,包括您所说的更改。但是,问题仍然存在。
  • 你为什么还要这样做?您的desenfileirar() 函数收到一个指向列表的指针。它分配一个新结构,将列表的头部复制到新结构中,然后将列表重新指向下一个元素,并释放头部。它还有一个内存泄漏。为什么不节省自己的精力,并消除分配/复制步骤? FILE *desenfileirar(FILA **F) { FILA *P; if (vazia(F)) {return NULL;} else {P=*F; *F=(*F)-&gt;prox; return P;}}(哦,是的:你在哪里采样输入数据?)
  • 那里。 更更简单,并消除了一个潜在的内存泄漏。 (当然,你还有其他人......)
  • 这是评估释放和内存分配的一部分。我需要释放未排队的元素。所以,我需要将它复制到一个 FILA 类型的指针并返回这个指针。我知道这是浪费内存,但是,如果我想要 100% 的分数,我需要这样做。 teste.txt 是:“0 1 0 0 0 0 0 0 1 0 0 1 1 0 0 0 1 1 0 1 0 0 1 0 0” 注意:它是一个二维数组 5x5
猜你喜欢
  • 1970-01-01
  • 2022-12-04
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2022-06-10
  • 1970-01-01
  • 2012-04-20
相关资源
最近更新 更多