函数返回C中局部变量错误的地址

Question

我有以下代码：

 char* gen()
 {
     char out[256];
     sprintf(out, ...); // you don't need to know what's in here I don't think

     return out;
 }

当我尝试编译时出现此错误：

ERROR: function returns address of local variable

我试过让这个返回char[]和char没有运气。 我错过了什么吗？

Answer 1

你的char数组变量out只存在于功能的体内。
当您从函数返回时，不能再访问out缓冲区的内容，它只是函数的本地内容。

如果您想从函数返回一些字符串给调用者，您可以在函数内部动态分配该字符串（例如使用malloc() ）并将指向该字符串的指针返回给调用者，例如

char* gen(void)
{   
    char out[256];
    sprintf(out, ...);

/* 
 *   This does NOT work, since "out" is local to the function.
 *
 *   return out;
 */

    /* Dynamically allocate the string */
    char* result = malloc(strlen(out) + 1) /* +1 for terminating NUL */

    /* Deep-copy the string from temporary buffer to return value buffer */
    strcpy(result, out);

    /* Return the pointer to the dynamically allocated buffer */
    return result;
    /* NOTE: The caller must FREE this memory using free(). */
}

另一个更简单的选择是将out缓冲区指针作为char*参数以及缓冲区大小（以避免缓冲区溢出）传递。

在这种情况下，您的函数可以直接将字符串格式化为作为参数传递的目标缓冲区：

/* Pass destination buffer pointer and buffer size */
void gen(char* out, size_t out_size)
{   
    /* Directly write into caller supplied buffer. 
     * Note: Use a "safe" function like snprintf(), to avoid buffer overruns.
     */
    snprintf(out, out_size, ...);
    ...
}

请注意，您在问题标题中明确指出了“C”，但您添加了[c++]标签。 如果您可以使用 C++，最简单的做法是使用像std::string这样的字符串类（并让它管理所有字符串缓冲区内存分配/清理）。

Answer 2

当您在函数char out[256];使用以下声明时char out[256]; 一旦函数返回，分配的空间将被释放，因此返回指向out char数组的指针是没有意义的。

如果你想返回一个指向你在函数中创建的字符串的指针，你应该使用malloc()作为

char* out = (char*)malloc(256*sizeof(char));

它为 256 个char分配空间，但应该在某个时候使用free()函数手动释放。

或者按照Brian Bi的评论中的建议，传递一个char *指向要用作函数参数的字符串。

Answer 3

问题是当函数gen返回（退出）时，它的局部变量（例如out ）超出了作用域并且调用者不再可以访问它。 因此，当您返回out您会返回一个指向不再分配的内存的指针。

从函数“返回”指针/缓冲区有两种选择：

1. 在调用函数中分配缓冲区并将其传递给gen ：

    char out[256]; gen(out, sizeof out);

   通常还提供您传入的缓冲区的大小，因为被调用的函数无法知道这一点。 这意味着您必须将 gen 的声明更改为：

    void gen(char * out, size_t size){

   您还可以将传入缓冲区的大小硬编码为 256（因为您现在已在gen函数中对其进行了硬编码）：

    void gen(char out[256]){

   这意味着您必须向gen提供一个char[256]类型的变量（并且没有其他指针或数组）。 但它确实允许您在gen内部执行sizeof out 。

2. 在函数内部动态分配缓冲区：

    char * out = malloc(256 * sizeof *out); // ... return out;

   这样做的好处是gen的声明不会改变。 但这确实意味着调用函数在完成后必须free返回的缓冲区。