sprintf 的分段错误

Question

我正在创建一个 makefile 创建器，但我在 sprintf 上遇到了这个错误，最奇怪的是我在出现错误的一个之前有多个 sprintf，它们工作正常。

代码如下：

if ( WIFEXITED(stat)  ){

    if ( WEXITSTATUS(stat) ) {

        if ( cFiles == 0 && cFolders == 0 ) {
            Crear(path);
        }

        cFolders = 1;
        TEMP = malloc( sizeof(char)*( strlen(direntp->d_name) + 25 ) );

        if ( TEMP == NULL ) {
            perror("Malloc Error: ");
            exit(1);
        }

        if ( sprintf(TEMP, "\n%s/%s.a: force\n\t$(MAKE) -C %s\n",direntp->d_name, direntp->d_name, direntp->d_name) < 0 ) {
        perror("Sprintf Error: ");
        exit(1);
        }

        write(STDOUT_FILENO,TEMP,strlen(TEMP));
        f.name = malloc( sizeof(char)*( strlen(direntp->d_name)*2 + 3 ) );

        if ( f.name = NULL ) {
            perror("Malloc Error: ");
            exit(1);
        } 
            //This is the one with the problem!!!       
            if ( sprintf(f.name, "%s/%s.a", direntp->d_name, direntp->d_name) < 0 ) {
            perror("Sprintf Error: ");
            exit(1);
        }

        l = AddToList(l,&f);
    }
}

Answer 1

您似乎分配的 TEMP 不够大

TEMP = malloc( sizeof(char)*( strlen(direntp->d_name) + 25 ) );

应该是：

TEMP = malloc( sizeof(char)*( strlen(direntp->d_name)*3 + 25 + 1 ) );

您正在打印 d_name 三次，并且您还需要一个额外的字节作为空终止符。

同样，在这一行：

   f.name = malloc( sizeof(char)*( strlen(direntp->d_name)*2 + 3 ) );

应该是

   f.name = malloc( sizeof(char)*( strlen(direntp->d_name)*2 + 3 + 1) );

考虑空终止符。

使用这样的断言有助于确保您的计算是正确的：

    int TEMP_size = strlen(direntp->d_name)*3 + 25 + 1; 
    TEMP = malloc(sizeof(char)*TEMP_size);

    if ( TEMP == NULL ) {
        perror("Malloc Error: ");
        exit(1);
    }

    if ( sprintf(TEMP, "\n%s/%s.a: force\n\t$(MAKE) -C %s\n",direntp->d_name, direntp->d_name, direntp->d_name) < 0 ) {
      perror("Sprintf Error: ");
      exit(1);
    }
    assert(strlen(TEMP)+1==TEMP_size);

Answer 2

“%s/%s.a”，所以我认为如果“%s”的长度为 n，那么整个字符串将采用 n*2+1+1+1+1=2n+4 而不是 2n+3 ...不要忘记终止的 '\0',,, 但是至于这是否会导致分段错误...我不知道...

//是的，这段代码中有很多关于分配字节数的错误......

Answer 3

您没有为此 sprintf 分配足够的内存：

sprintf(TEMP, "\n%s/%s.a: force\n\t$(MAKE) -C %s\n",direntp->d_name, direntp->d_name, direntp->d_name

你分配 sizeof(direntp->d_name) +25 但我看到 25 char + 3* sizeof(direntp->d_name) +1 extra char（你的 \0 表示你的字符串的结尾）

问候

Answer 4

除了以前的答案之外的一些注释：

1. 大多数现代类 Unix 系统在标准库中都有 asprintf() 和 vasprintf()，它们自己分配缓冲区。使用它们比计算所需大小、分配和调用 sprintf() 更简单，即使它们过度分配了一点。

2. 填充缓冲区然后使用 write() 将其打印到 stdout 看起来比简单的 printf() 复杂得多。除非您使用 stdio 无法稳定处理的东西（如非阻塞 I/O），否则将 stdio 用于此类任务要好得多。

3. 您正在执行的任务更适合某些脚本语言（Perl、Python、Tcl、Ruby 等），除非某些外部奇怪的情况迫使您使用 C； C 应该以 Unix 的方式实现更接近内核和低级的区域。参见例如"The Art of Unix Programming"推理。