释放分配给函数的指针？

Question

如果需要将未签名的char *传递给函数，什么时候应该释放我的未签名的char *？ 例

void myFunction(unsigned char *request){
   // do I need to use free(request); here?
}  

 int main (){       
 // main code 

 unsigned char *request = NULL;
 // read from buffer and reallocate memory (using realloc & memcpy)
 myFunction(request); // I do not want to read or use "request" after this function call. 
 free(request); // is this ok? Or should this be inside myFunction also?

 return 0; 
 }

Answer 1

完成使用后，立即对某物使用free() 。 因此，例如，您可能不会在myFunction()执行此操作，因为当您将该函数返回给main时，您可能仍然会对request指向的值感兴趣。

Answer 2

通常，您必须在分配它的位置释放它，而不需要更多它。 在这种情况下，您可以在两个地方都可以执行此操作，也可以在释放后将其设置为NULL，并且下次尝试释放它时，请检查是否为null

你也可以实现类似的东西：

void FreeAndNull(void *request){
   if request <> NULL then 
     { free(request); request = NULL; }
}

Answer 3

您不必执行此操作-但一个好的做法是在分配内容的同时释放它们。 例如，不要在函数中分配某些东西并在返回后释放它。 因为那样，几个月后再次查看代码时，内存的生命周期就变得不清楚。

因此，执行以下操作要好得多：

request = malloc();
doStuff( request );
free( request );

比你所拥有的模式。 C ++通过创建构造函数和析构函数来帮助您使用此模式。 也就是说，构造函数实际上可能会分配内存，并实质上以对象的形式将其返回给您，但是编译器会为您释放内存而自动调用析构函数。

因此，如果将请求缓冲区包装在类中，则可以使用更安全，更可维护的模式来实现所需的模式，例如：

Request::Request() { mRequest = malloc(); }
Request::~Request() { if ( mRequest ) free( mRequest ); mRequest = NULL; }

...

{
    Request request;
    myFunction( request );
}

使用此模式，您的内存将自动清除。 更安全，更易维护。