为什么我不能使用动态分配的字符串创建临时文件?

时间:2015-10-24 13:21:45

标签: c linux mkstemp

我正在尝试使用mkstemp创建一个包含日志条目的目录。但是,根据我的理解,我不能将字符串常量传递给mkstemp。我为字符串分配内存,并使用snprintf格式化我认为可以工作的输出,但mkstemp返回负值设置errno为EINVAL。

然而,在mkstemp的{​​{1}}中,它明确地说:

  

EINVAL对于mkstemp()和mkostemp():最后六个字符     模板不是XXXXXX;现在模板没有变化。

Furhtermore mkstemp永远不会修改我的动态字符串。

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

#define LOG_DIR "/tmp"

int main(int argc, char **argv) {
  char *fname;
  FILE *fp;

  if(argc != 3) {
    fprintf(stderr, "Usage: %s <msg> <severity>\n", argv[0]);
    return 0;
  }

  int length = snprintf(NULL, 0, "%s/log_entry.XXXXXX", LOG_DIR); // snprintf returns the required length for my string
  if(length < 0) {
    perror("snprintf failed");
    return 1;
  }
  fname = malloc(sizeof(char) * length); // allocate memory for fname based on the return value of snprintf
  snprintf(fname, length, "%s/log_entry.XXXXXX", LOG_DIR); // send formatted output into fname

  int fd = mkstemp(fname); // this returns -1 and errno is set to 22
  if(fd < 0) {
    perror("failed to create entry file");
    return 1;
  }
  fp = fdopen(fd, "w");
  if(fp == NULL) {
    perror("failed to open entry file");
    return 1;
  }
  fprintf(fp, "\"%s\" %d ",argv[1], atoi(argv[2]));
  fflush(fp);
  fclose(fp);
  free(fname);
  return 0;
}

这个代码片段在我的两台Linux机器上都会出错,但是如果我删除动态分配的字符串并明确设置fname它就可以了

char fname[] = "/tmp/log_entry.XXXXXX";

2 个答案:

答案 0 :(得分:2)

fname = malloc(sizeof(char) * length);

应该是:

fname = malloc(sizeof(char) * (length + 1)); 

现在

snprintf(fname, length+1, "%s/log_entry.XXXXXX", LOG_DIR); 

将创建文件名。在您的版本中,文件名不会以6&#39; X结尾,导致mkstemp失败。

答案 1 :(得分:1)

 fname = malloc(sizeof(char) * length); 

您完全填写,'\0'不留空间。 :为null终止符提供空格 -

 fname = malloc(sizeof(char) *(length+1)); 

然后将snprintf的长度也增加到length+1