C语言拼接两个char*字符串报错，求错误原因及解决方法

原代码 #

char* deviceId = "A1B2C3D4";
char* channelUrl = "/arduino/subscribers/";


char* getSubscriberUrl(char* channelUrl, char* deviceId) {
  char* buffer[sizeof(channelUrl) + sizeof(deviceId)+2];
  strcat(buffer, channelUrl);
  strcat(buffer, deviceId);

  return buffer;
}

编译错误信息 #

initializing argument 1 of 'char* strcat(char*, const char*)'
 char  *strcat (char *__restrict, const char *__restrict);
                ^~~~~~
sketch_sep13a:87:10: error: cannot convert 'char**' to 'char*' in return
   return buffer;

代码存在的错误点 #

• 缓冲区类型声明错误：char* buffer[...]; 声明的是一个字符指针数组（类型为char**），但strcat需要的是char*类型的字符数组作为目标缓冲区。正确的声明应该是char buffer[...];（去掉指针符号*）。
• 字符串长度计算错误：sizeof(channelUrl)和sizeof(deviceId)获取的是指针本身的字节大小（比如32位系统是4字节，64位是8字节），不是字符串的实际长度。需要用strlen(channelUrl)和strlen(deviceId)来获取字符串的有效长度，才能计算出足够容纳拼接后字符串的缓冲区大小。
• strcat使用逻辑错误：strcat会从目标缓冲区的\0位置开始追加内容，但这里的缓冲区未初始化，没有初始的\0，直接调用strcat会导致未定义行为。正确的做法是先将第一个字符串复制到缓冲区（用strcpy），再追加第二个字符串（用strcat）。
• 返回局部变量的野指针问题：就算缓冲区声明正确，它是函数内的局部自动变量，函数执行完毕后局部变量会被销毁，返回的指针会变成野指针，后续访问该指针指向的内存会导致程序崩溃或异常行为。如果需要返回拼接后的字符串，应该使用动态分配内存（比如malloc），或者让调用者提供缓冲区。

内容的提问来源于stack exchange，提问作者Itzik.B