fopen/fclose 上的段错误

Question

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

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, "Error closing %s. Exiting...\n", name);
        exit(1);
    }
}

当我到达 WriteFile1() 中的第一个 fprintf 语句时，它就会出现段错误。我是否对 FILE 变量做了错误的操作？看起来它应该像以前一样工作。唯一的区别是文件名字符串的 malloc，我将其作为名称传递并在引号中给出实际值。

谢谢

Answer 1

这段代码是错误的：

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

这里您只是分配给本地 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;
}

Answer 2

您的 open 函数应如下所示：

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 * 实际上是函数的局部变量（因为它是参数）。在函数中更改它不会在外部世界中更改它。

在设计返回指针（或任何其他相关内容）的函数时，始终更喜欢通过 return 语句返回指针，而不是尝试通过参数列表来返回指针。

Answer 3

这：

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 始终为 true，因此根本没有必要涉及它。