Question

我知道什么是分段错误，我不需要知道它的定义:) 我只需要知道它在我的代码中的来源。此程序用于将单词作为输入，从文本文件中读取，写入单独的文本文件，然后打印读取文件和输入中的所有单词。

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

int main(int argc, char*argv[]){
    FILE *read;
    FILE *write;
    char **list=malloc(sizeof(char*));
    char **oldlist=malloc(sizeof(char*));
    char *oldword=malloc(sizeof(char));
    char exit[]="end";
    int a, c, r=0, w=0, n=0, z= 0, y=0, d=0, g=0;

//check arg
for(a=0; a<argc; a++){
    if(strcmp(argv[a], "-r")==0){
        r =1;
        read=fopen("read.txt", "r");
    }
    else if(strcmp(argv[a], "-w")==0){
        w =1;
        write=fopen("write.txt", "w");
    }   
}
if(r==0 && w==0){
    printf("Error: Invalid Command.\n");
}
printf("Read = %d | Write = %d\n", r, w);

//getwords
printf("Enter your words: ");
while(1){
char *word=malloc(sizeof(char));
list=realloc(list, sizeof(char*)*(z+10));
word=realloc(word, sizeof(char)*(z+10));
scanf("%s", word);
              if (strcmp(word,exit)==0){
        break;
              }
              else{
                *(list+z) = word;
            z++;
          }
}   

//read
if (r==1){
    do{
        while(1){
            *(oldword+d)=fgetc(read);
            d++;
        }
    }while(feof(read) != 0);            
}
*(oldword+(d-1))="\0";

printf("Your words are:\n");
            puts(oldword);
    for(c=0; c<n; c++){
            puts(*(list+c));
    }

    //write
    if (w ==1){
        if(w==1){
            fputs(*(oldlist+c),write);
        }
    for(c=0; c<n; c++){
            fputs(*(list+c),write); 
        }
    }
    //end
    free(list);
    fclose(read);
    fclose(write);          
    return 0;

}

Answer 1

为单词分配1个字节：

char *word=malloc(sizeof(char));

然后你读了一个字符串。这是一个缓冲区溢出，并导致很大的不满（和Stack Overflow上的这样的问题）。具体来说，读取长字会占用“堆”（由malloc()等控制的数据空间）中的控制信息和用于确定哪些空间可供使用而哪些空间不可用的加扰数据。什么构成“长字”取决于系统;从技术上讲，除了空字符串（只是一个终端'\0'）之外的任何字符串都太长了，但是如果单词是7个字节或更少，你可能会逃避蓝色谋杀。

但是realloc不应该处理内存问题吗？我正在尝试使字符串输入无限制。

嗯...作为一种开展业务的方式有点奇怪，但在你实际使用之前realloc() word可以解决一些问题。

具体来说，在循环第一次溢出word的内存分配之前，您可以读取最多包含9个字符。然而，这使它成为“无限制”还有很长的路要走，让它无限制是非常重要的。一个问题是%s停止扫描空格字符;总的来说，这对你有利。另一个原因是你似乎使用z作为你输入的字符串数量的计数，也可以作为你可以输入的字符串的长度。如果列表超过10个条目，则不会重新分配list。

您仍然可能遇到各种问题。例如，您对oldword的处理不会执行额外的realloc（）;它只分配一个字节。然后你有一个完全无界的无限循环（如果它完全输入的话）。这个循环是一场灾难：

    while(1){
        *(oldword+d)=fgetc(read);
        d++;
    }

Answer 2

如果使用debugging（g标志）进行编译并在valgrind下运行，它应该可以很好地指示它为什么会出错。

C分段故障中的文件I / O.

2 个答案: