释放分配的字符串时内存损坏

Question

当我尝试运行该程序时，我收到错误 malloc(): memory corruption。 该错误并非直接来自此函数，它发生在我尝试在此函数之后执行 malloc() 时。如果我删除了 free(ch) 行，它可以正常工作，所以我猜想当我尝试释放它时会发生损坏。 main() 是我如何使用该函数的示例。

char * makeInt(int val){ 
  char *res = malloc(5);
  char l [5] = "";
  sprintf(l,"%d",val);
  if(val < 10){
    strcat(res,"000");
    strcat(res,l);
  }
  else if(val < 100){
    strcat(res,"00");
    strcat(res,l);
  }
  else if(val < 1000){
    strcat(res,"0");
    strcat(res,l);
  }
  else if( val < 10000){
    strcat(res,l);
  }
  res[4] = '\0';
  return res;
}


  char * makeString(char *ch){
  int t = strlen(ch);
  char *chaine = malloc(t+4);
  char *nb = makeInt(t);
  strcat(chaine,nb);
  strcat(chaine,ch);
  chaine[t+4] = '\0';
  free(ch);
  return chaine;
}

int main(){
    char *path = malloc(100);
// here we do many operations on path, when i call makeString, path contains something
        path = makeString(path);
    }

编辑：抱歉，我发布的时候太晚了，我忘记了一些信息。 我添加了 makeInt()。关于包含，我将它们包含在我的代码中，但我认为错过的包含不会导致内存损坏，因为它会编译。此外，当我调用 makeString() 时，路径包含一个字符串。我在代码中的不同位置使用 makeString()。当我添加 free(ch) 时出现错误，但我不明白为什么释放 main 中分配的内存会导致内存损坏。

Answer 1

贴出的代码包含一定的逻辑错误，例如：

strcat(chaine,nb);

但初始字符串必须有一个 NUL 字节，否则不知道会返回什么值（即未定义的行为）

malloc() 的返回值在第一个字符中可能有也可能没有 NUL 字节。

这可能是发布的代码导致段错误事件的原因。

（您可以通过使用calloc() 而不是malloc() 来解决这个特定问题

另外，makeString() 的参数未初始化为任何特定的以 NUL 结尾的字符串。所以将该参数传递给strlen() 是未定义的行为

（因为遇到的第一个 NUL 字节很可能超出了 'path 数组的末尾）

一些建议：

1. 阅读该代码使用的任何系统函数的手册页。
2. 单步执行任何代码（最好使用调试器）以查看其实际作用。

这里是可能适合您的代码版本。

#include <stdio.h>  // printf(), sprintf()
#include <stdlib.h> // malloc()
#include <string.h> // strlen(), strcat()

// prototypes
char * makeString(char *ch);


char * makeString(char *ch)
{
    // EDIT: this line was wrong: char *chaine = malloc(sizeof(ch) + 7)  // large enough to handle any int value
    char *chaine = malloc(strlen(ch) + 7);

    sprintf( chaine, "%4lu", strlen( ch ) );
    strcat( chaine, " " );
    strcat( chaine, ch );

    return chaine;  // the caller must call `free()`
} // end function: makeString


int main( void )
{
    char *path = "my path string";
    path = makeString(path);
    printf( "%s\n", path );
    free(path);
} // end function: main

输出是：

  14 my path string