getc函数生成segfault

Question

程序：

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

char *f_gets(char *s, int n, FILE *iop)
{
    int c=0;
    char *cs;
    cs = s;

    while (--n > 0 && (c = getc(iop)) != EOF)
    {
        if ((*cs++ = c) == '\n')
            break;
    }
    *cs = '\0';
    return (c == EOF && cs == s) ? NULL : s;
}


main(int argc, char *argv[])
{
    FILE *fp1,*fp2;
    char s2[100],s1[100];
    if (argc <= 2 )
        printf("2 argument needed \n"); 
    else
        if((fp1=fopen(argv[1],"r"))== NULL && (fp2=fopen(argv[2],"r"))==NULL)
            printf("cat: can't open The file\n");
        else
        {
            while(1)
            {
                f_gets(s1,100,fp1); // 1st iteration
                f_gets(s2,100,fp2); // 2nd iteration
                if(!strcmp(s1,s2)) 
                    printf("%s %s",s1,s2);
            }
            fclose(fp1);
            fclose(fp2);
        }
}

输出：

$ ./a.out a b
Segmentation fault (core dumped)
$ 

在上面的程序中，当我们第二次调用f_gets时，就会发生段故障。即使我两次检查程序，也很难找到问题所在。有没有人解释为什么它会产生这个问题。

Answer 1

当您进行调用时，您的第二个文件未打开。

问题是，您正在使用短路从路径调用fopen：

if((fp1=fopen(argv[1],"r"))== NULL && (fp2=fopen(argv[2],"r"))==NULL)

由于代码中的错误，当fp1正常打开时，fp2将始终保持关闭状态。这是因为(fp1=fopen(argv[1],"r"))== NULL的计算结果为0，并确保(fp2=fopen(argv[2],"r"))==NULL永远不会被调用。

您可以通过将&&替换为||来解决此问题，但更好的方法是一次打开一个文件。

Answer 2

if((fp1=fopen(argv[1],"r"))== NULL && (fp2=fopen(argv[2],"r"))==NULL)

表示“如果程序无法同时打开fp1和fp2”。你可以通过使用||来解决这个问题。但最好不要编写又长又复杂的表达式，因为它们只会增加编写bug的可能性。

相反，请执行以下操作：

fp1=fopen(argv[1],"r");
if(fp1 == NULL)
{
  // error handling
}

fp2=fopen(argv[2],"r");
if(fp2 == NULL)
{
  fclose(fp1);
  // error handling
}

作为奖励:仅仅因为我们将程序重写为可读的内容，我们现在还发现了第二个bug。如果程序打开fp1但无法打开fp2，它在中止之前从未关闭过fp1。