c - 有时我的程序会出现段错误，但有时它可以工作，但我找不到失败的地方

Question

已关闭。此问题需要 debugging details 。目前不接受答案。

---

编辑问题以包含 desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem 。这将有助于其他人回答问题。

已关闭 5 年前。

Improve this question

我正在学习c，所以我不太了解它。编译代码时没有失败或错误，但当我尝试运行它时，有时它只是失败，给我一个 segmentation fault 。我尝试在 GDB 中运行它，但没有给我带来任何问题，它总是工作得很好。

#include <stdlib.h>
#include <stdio.h>
#include <string.h>

typedef struct{
    int palabra;
    int cantPag;
    int *paginas;
} palPag;

typedef struct{
    int nConsulta;
    int uno;
    int dos;
} stConsulta;

void consultarPaginas(palPag *arrayPalabras, stConsulta *arrayConsultas, int cantConsultas, int cantPalabras, FILE *salida){
    int m;
    for (m = 0; m < cantConsultas; m++){
        int n, o, p, q, r, s, min;
        n = arrayConsultas[m].uno;
        o = arrayConsultas[m].dos;
        p = 0;
        q = 0;
        r = 0;
        if (arrayPalabras[n].cantPag < arrayPalabras[o].cantPag){
            min = arrayPalabras[n].cantPag;
        }
        else{
            min = arrayPalabras[o].cantPag;
        }
        int *arrayAux = (int *) malloc(sizeof(int)*min);
        while (p < arrayPalabras[n].cantPag && q < arrayPalabras[o].cantPag){
            if (arrayPalabras[n].paginas[p] < arrayPalabras[o].paginas[q]){
                p = p+1;
            }
            else if (arrayPalabras[n].paginas[p] > arrayPalabras[o].paginas[q]){
                q = q+1;
            }
            else if (arrayPalabras[n].paginas[p] == arrayPalabras[o].paginas[q]){
                arrayAux[r] = arrayPalabras[n].paginas[p];
                r = r+1;
                p = p+1;
                q = q+1;
            }
        }
        fprintf(salida, "%d", r);
        printf("%d\n", r);
        for (s = 0; s < r; s++){
            fprintf(salida, " ");
            fprintf(salida, "%d", arrayAux[s]);
        }
        fwrite("\n", 1, sizeof(char), salida);
        free(arrayAux);
    }
}

void sort(palPag *arrayPalabras, int cantPalabras) {
    int x;
    for (x = 0; x < cantPalabras; x++){
        int i, j, a, n;
        n = arrayPalabras[x].cantPag;
        for (i = 0; i < n; ++i){
            for (j = i + 1; j < n; ++j){
                if (arrayPalabras[x].paginas[i] > arrayPalabras[x].paginas[j]){
                    a =  arrayPalabras[x].paginas[i];
                    arrayPalabras[x].paginas[i] = arrayPalabras[x].paginas[j];
                    arrayPalabras[x].paginas[j] = a;
                }
            }
        }
    }
}

int main(){
    FILE *palabras, *consultas, *salida;
    char line[100];
    palPag *arrayPalabras;
    stConsulta *arrayConsultas;
    int cantPalabras, cantConsultas, j;

    palabras = fopen("palabras.dat", "r");
    if (palabras == NULL){
        printf("El archivo palabras.dat no se ha leído correctamente\n");
    }

    consultas = fopen("consultas.dat", "r");
    if (consultas == NULL){
        printf("El archivo consultas.dat no se ha leído correctamente\n");
    }

    if ((palabras == NULL) || (consultas == NULL)){
        printf("Alguno de los archivos no ha podido ser leído\n");
        exit(0);
    }
    salida = fopen("salida.dat", "w");

    //Stream al inicio del archivo (provisional)
    fseek(palabras, 0, SEEK_SET);
    fseek(consultas, 0, SEEK_SET);

    while(fgets(line, sizeof(line), palabras) != NULL){
        cantPalabras++;
    }

    while(fgets(line, sizeof(line), consultas) != NULL){
        cantConsultas++;
    }

    //Stream al inicio del archivo
    fseek(palabras, 0, SEEK_SET);
    fseek(consultas, 0, SEEK_SET);

    //Arreglo de palabras y sus páginas
    arrayPalabras = (palPag *) malloc(sizeof(palPag)*cantPalabras);

    for (j = 0; j < cantPalabras; j++){
        int k;
        int l;
        arrayPalabras[j].palabra = j;
        fscanf(palabras, "%d", &k);
        arrayPalabras[j].cantPag = k;
        arrayPalabras[j].paginas = (int *) malloc(sizeof(int)*k);
        for (l = 0; l < k; l++){
            fscanf(palabras, "%d", &arrayPalabras[j].paginas[l]);
        }
    }

    //Arreglo de consultas
    arrayConsultas = (stConsulta *) malloc(sizeof(stConsulta)*cantConsultas);

    for (j = 0; j < cantConsultas; j++){
        int k;
        int l;
        arrayConsultas[j].nConsulta = j;
        fscanf(consultas, "%d %d", &k, &l);
        arrayConsultas[j].uno = k;
        arrayConsultas[j].dos = l;
    }

    sort(arrayPalabras, cantPalabras);
    consultarPaginas(arrayPalabras, arrayConsultas, cantConsultas, 
    cantPalabras, salida);

    free(arrayConsultas);
    free(arrayPalabras);
    fclose(salida);
    fclose(palabras);
    fclose(consultas);
    return(0);
}

正如我所说，我不太懂c，所以我的代码有点乱，而且还没有完成。

Answer 1

原因之一可能是使用了未初始化变量。

在 main你没有初始化cantPalabras使用前。这是未定义的行为。

cantConsultas 也是如此

另一个好的原因是:

    fscanf(palabras, "%d", &k);
    arrayPalabras[j].cantPag = k;
    arrayPalabras[j].paginas = (int *) malloc(sizeof(int)*k);

您没有检查 fscanf 的返回值所以你不知道是否 k已初始化。换句话说，k可以在未初始化的情况下使用。

你应该这样做:

    if (fscanf(palabras, "%d", &k) != 1)
    {
        // Add error handling here
        ...
    }
    arrayPalabras[j].cantPag = k;
    arrayPalabras[j].paginas = (int *) malloc(sizeof(int)*k);