这是否使用C指针可以防止泄漏内存？

Question

char *a() { 
    char *t = malloc(8); 
    t[0] = 'a'; 
    t[1] = 'b'; 
    //... 
    t[7] = 'h'; 
    return t; 
} 

int main(void) { 
    char *x = a(); 
    //do something with x 
    //... 
    free(x); 
    return 0; 
} 

此代码是否有任何潜在的问题，因为内存分配在a()和已用内存main()？

Answer 1

不，您明确不会有任何问题，因为您分配了a中的功能并在main中使用它。

Answer 2

没有问题—其实，这是分配函数返回—但你可能要free()内存当你用它完成后，将仍然使用的内存的正确方法。八个字节不会成为问题，但针对内存泄漏进行编码是一个很好的习惯。

Answer 3

只要你记得释放x指向的内存，那么你将不会有任何问题。通过使用malloc（），您可以在堆回到调用函数时独立分配内存。

Answer 4

首先，a（）被声明为返回void，但您试图返回一个char *。更改签名以返回char *。

其次，你的函数很好，但是你的示例代码有内存泄漏，因为你永远不会释放返回的指针指向的内存。

第三，正如gbrandt指出的那样，在调用malloc之后，您没有检查成功。 malloc可能会失败，并检查它是否成为一个好习惯。

---

另一种方式来实现，这将是一个指针传递给一个指针变成（）来代替，然后调用者有其传递到（）之前创建的指针本身，但无论哪种方式，你仍然需要释放内存。说实话，在这种情况下，我会用你的方法来解决这个问题。没有令人信服的理由这样做，只是想我会提到它。

void a(char **p) 
{ 
    *p = malloc(8); 
    if (*p) 
    { 
     **p[0] = 'a'; 
     **p[1] = 'b'; 
     ... 
     **p[7] = 'h'; 
    } 
} 

int main(void) 
{ 
    char *x; 
    a(&x); 
    //do something with x 
    ..... 
    free(x); 
} 

如果这种替代方法混淆了你，请让我知道，我会很乐意提供解释（虽然，此刻，我要回去工作！）

Answer 5

除了有a()（应该是char *而不是void）的错误返回类型，代码没有任何问题。

只要确保将free()分配给内存即可。

Answer 6

以上所有的好建议。代码的小问题，最大的问题是...

您不在malloc中检查成功或函数调用。不要忘记错误处理。

Answer 7

这里的大多数答案表明这不是问题，只记得稍后释放（）它。这在技术上是正确的，但确实是一个问题。任何时候你在一个范围内分配内存，并期望它在另一个范围内释放，你就会要求内存泄漏。人们不记得释放记忆。还有一个问题，调用者必须知道你使用了malloc，而不是alloca（）或new（）或其他东西。如果他们调用除匹配的解除分配例程之外的其他任何内容，则结果是不确定的。

总而言之，分配内存并将其传回给调用者通常是一个错误。

最好是期望内存由调用者分配，因为如果他们分配内存，他们会记住释放内存并正确执行。

Answer 8

谢谢。我刚编辑了代码。我想这次应该没有问题。顺便说一下，在C中，当函数完成时，编译器是否清除堆栈中的所有变量（包括指针012）？