c - 多次执行后fgets()崩溃

我正在编写一个程序来破解CRC16。我在输出文件和保留计算出的CRC16时遇到了一些问题（不知道为什么将其写入文件时它会更改）。因此，我在这里要做的是读取输入文件，将其写入带有一点乱码的输出文件中，然后再次读取输出文件并计算其CRC16。如果它与所需的CRC16匹配，则完成。但是，在一堆执行之后，fgets（）方法因Seg错误而崩溃。

有人可以帮助我吗？请忽略性能问题，这是一个测试。

int main(int argc, char* argv[]){

        char outfile[strlen(argv[1])];
        strcpy(outfile,argv[1]);

        strcat(outfile,".crack");

        char crc16[5];
        strcpy(crc16,argv[2]);
        char newcrc16[5];
        char gebrish[80];
        char cat[2];
        int full = 1;
        int p = 0;
        int i,j,k;


        for(i=32; i< 128;i++)
                for(j=32; j< 128; j++)
                        for(k=32; k < 128; k++){
                                gebrish[0] =i;
                                gebrish[1] =j;
                                gebrish[2] =k;
                                gebrish[3] = '\n';
                                gebrish[4] ='\0';

                                boost::crc_16_type result;

                                FILE* file;
                                FILE* out;
                                char line[100];

                                printf("read out\n");
                                out = fopen(outfile,"w");

                                printf("read file\n");
                                file = fopen(argv[1],"r");
                printf("wrt\n");
                                while(fgets(line,80,file) != NULL){
                                        fputs(line,out);
                                }
                                fputs(gebrish,out);

                                fclose(file);
                                fclose(out);

                                printf("read gain\n");
                                out = fopen(outfile,"r");

                                while(fgets(line,80,out) != NULL){
                                        result.process_bytes(line,strlen(line));
                                        printf("%s",line);
                                }

                                int crc = result.checksum();

                                sprintf(newcrc16,"%x",crc);
                                printf("%s",newcrc16);

                                if(strcmp(crc16,newcrc16) == 0){

                                        printf("%s",gebrish);
                                        return 0;

                                }
                        }



        return 0;
}

最佳答案

这会导致缓冲区溢出：

char outfile[strlen(argv[1])];
strcpy(outfile,argv[1]);

strcat(outfile,".crack");

因为outfile中没有足够的空间来终止空字符和".crack"。它将覆盖原本不应该的内存，并且可能是分段错误的原因。

改成：

    char outfile[strlen(argv[1]) + 1 + 6];
    strcpy(outfile,argv[1]);

    strcat(outfile,".crack");

在访问argv元素之前，请通过检查argc确保已提供了它们：

if (argc > 2)
{
    /* Safe to use argv[1] and argv[2]. */
}

还检查fopen()中的返回值。

关于c - 多次执行后fgets()崩溃，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/10034912/