c - 为什么我在这里遇到段错误？ [C]

Question

这会得到反对票，但只要有人能帮助我，我就不在乎。我无法访问调试器，如果我这样做的话，我可能在几个小时前就完成了。

为什么我会在这里遇到段错误？这是否意味着我正在尝试访问一些不属于我的进程的内存位置？在将数据从一个文件复制到下一个文件之前，我是否应该实现一个缓冲区(尽管我不知道为什么这很重要)？

#include <stdio.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <unistd.h>
#include <errno.h>

int error( char * msg )
{
    perror( msg );
    return 2;
}

int usage( char * name )
{
    printf( "Usage: %s <file to copy> <name of copy>\n", name );
    return 1;
}

int main( int argc, char * argv[] )
{
    //making sure all required arguments have been passed
    if ( argc != 3 )
        return usage( argv[0] );

    //Make sure input exists, make sure output does not exist
    char *inputFname = argv[1];
    char *outputFname = argv[2];
    FILE *inputFile;
    FILE *outputFile;
    inputFile = fopen(inputFname, "r");
    int err = errno;
    outputFile = fopen(outputFname, "r");

    //handle file opening errors
    if(outputFile != NULL){
        return error("Error: target file already exists\n");
        fclose(outputFile);
    }
    if(inputFile == NULL && err == 2){
        //errno of 2 corresponds to error code ENOENT; no such file or directory
        return error("Error: file does not exist\n");
        fclose(inputFile);
    }
    if(inputFile == NULL && err ==  19){
        //errno of 19 corresponds to error code EACCE; permission error
        return error("Error: permission denied for file\n");
        fclose(inputFile);
    }
    fclose(outputFile);

    //no errors upon opening, gather info and make buffer
    printf("No errors, proceeding with copy");
    struct stat info;
    stat(inputFname, &info);

    //create new file
    outputFile = fopen(outputFname, "w");
    chmod(outputFname, info.st_mode & 07777);

    //copy the contents
    char nextChar;
    while( (nextChar = fgetc(inputFile)) != EOF)
        fputc(nextChar, outputFile);

    printf("copy completed");
    return 0;
}

Answer 1

1. fclose(outputFile);
   错误就在这里。如果 outputFile 会怎样？一片空白？即目标文件尚不存在(理想条件)？

   man -s3 fclose :

   The behavior of fclose() is undefined if the stream parameter is an illegal pointer.

<小时/>

从 main 返回后你觉得怎么样fclose会工作吗？

return error("Error: target file already exists\n");
fclose(outputFile);

<小时/>

如果 inputFile 会怎样？是 NULL和errno两者都不是2也不19 ？

提示:在这种情况下，前两个 if s 不会退出 main并且程序将继续执行文件错误检查代码 inputFile = NULL 。

<小时/>

此外，您还没有关闭复制完成的流。

<小时/>

我认为编写此代码的最佳方法是在打开输入文件后立即检查 inputFile 的错误。如果NULL然后退出否则继续。

然后类似地检查输出文件。如果不NULL然后退出否则继续。

所以正确的代码片段是:

...
inputFile = fopen(inputFname, "r");
int err = errno;

if(inputFile == NULL)
{ 
    if (err == 2)
        return error("Error: file does not exist\n");

    if(err ==  19)
        return error("Error: permission denied for file\n");

    return error("Other Error");
}

outputFile = fopen(outputFname, "r");
if(outputFile != NULL)
{
    fclose(inputFile);
    fclose(outputFile);
    return error("Error: target file already exists\n");
}  
...  
printf("copy completed");  
fclose(inputFile);
fclose(outputFile);
return 0;