Seg Fault在fopen / fclose上

问题描述

| 我有一个程序正在创建多个文件。每个创建的文件都有一个功能。每个函数中都有完全相同的代码来创建文件名，打开/创建文件以进行写入，设置其权限并最后关闭文件。我决定创建一个用于打开文件和关闭文件的函数，这样我就可以调用它，而不必每次都使用相同的代码。每个函数以前的代码如下所示：

void WriteFile1(char *name) {
   FILE *file;
   char *filename; //This is being malloc\'ed because it initially consisted of multiple strings

   if (!(filename = malloc(sizeof(char *) * (strlen(name) + 1)))) MallocError();
   if (!(file = fopen(filename,\"w\"))) {
       fprintf(stderr,\"Unable to open %s. Exiting \\n\",filename);
       exit(1);
   }
   fchmod(fileno(file),S_IRUSR | S_IWUSR | S_IXUSR | S_IRGRP | S_IXGRP | S_IROTH);

   //a bunch of fprintf statements here

   if (fclose(file)) {
       fprintf(stderr,\"Error closing %s. Exiting...\\n\",filename);
       exit(1);
   }
}

这工作得很好。我没有问题。现在看起来如下：

void WriteFile1() {
FILE *file;

OpenFile(file,\"filename.asdf\");
//fprintf statements
CloseFile(file,\"filename.asdf\");
}

void OpenFile(FILE *file,char *name) {
   if (!(file = fopen(name,\"w\"))) {
      fprintf(stderr,\"Unable to open %s. Exiting... \\n\",name);
      exit(1);
   }
   fchmod(fileno(file),S_IRUSR | S_IWUSR | S_IXUSR | S_IRGRP | S_IXGRP | S_IROTH);
}
void CloseFile(FILE *file,char *name) {
    if (fclose(file)) {
        fprintf(stderr,name);
        exit(1);
    }
}

当我进入WriteFile1（）中的第一个fprintf语句时，它就会出现故障。我使用FILE变量做错了什么吗？似乎它应该像以前一样工作。唯一的区别是文件名字符串的malloc，我改为将其作为名称传递并在引号中给出实际值。 谢谢     

解决方法

        这段代码是错误的：

void OpenFile(FILE *file,char *name) {
   if (!(file = fopen(name,\"w\"))) {

在这里，您只是分配给本地

file

变量。 您必须返回

file

，以便您的WriteFile1（）函数可以使用该FILE *

void WriteFile1() {
FILE *file;

file = OpenFile(\"filename.asdf\");
//fprintf statements
CloseFile(file,\"filename.asdf\");
}

FILE * OpenFile(char *name) {
   FILE * file;
   if (!(file = fopen(name,\"w\"))) {
      fprintf(stderr,\"Unable to open %s. Exiting... \\n\",name);
      exit(1);
   }
   fchmod(fileno(file),S_IRUSR | S_IWUSR | S_IXUSR | S_IRGRP | S_IXGRP | S_IROTH);
   return file;
}

    ,        您的打开函数应如下所示：

FILE *  OpenFile( char *name) {
   FILE * file;
   if (!(file = fopen(name,S_IRUSR | S_IWUSR | S_IXUSR | S_IRGRP | S_IXGRP | S_IROTH);
   return file;
}

在您的版本中，FILE *实际上是函数的局部变量（因为它是一个参数）。在功能中对其进行更改不会在外部环境中进行更改。 当设计返回指针的函数（或其他任何与此相关的函数）时，总是喜欢通过a7ѭ语句返回指针，而不是尝试通过参数列表来返回指针。     ,        这个：

filename = malloc(sizeof(char *) * (strlen(name) + 1))

应该：

filename = strdup(name);

如果有的话，否则类似：

if((filename = malloc(strlen(name) + 1)) != NULL)
{
   strcpy(filename,name);
   ...
}

请特别注意，每个字符只是一个

char

，而不是一个

char *

。由于

sizeof (char) == 1

始终是正确的，因此完全没有意义。