割り当てられた変数を返す

Question

私はmallocで割り当てられた変数を解放するべきだと知っていますが、それを関数に返すとどうなりますか？このようなもの：

char *somefunction(int somearg){ 
    char *str; 

    str=(char *)malloc(sizeof(char *)); 

    //some code 

    return str; 
} 

strを解放する必要がありますか？どうすればいい？

Answer 1

2つのオプションがあります：1つは、ある関数にchar *を渡し、ある関数内で割り当てる代わりに、または2つでは、後で戻り値を解放することです。

最初のオプション：

char *somefunction(char *str, int somearg){ 

    //some code 

    return str; 
} 

// Elsewhere... 
char *str = (char *) malloc....; 
somefunction(str, 123); 
// Code... 
free(str); 

番目のオプション：

char *somestr = somefunction(123); 
// Do something... 
free(somestr); 

それはarbitary関数内で割り当てられていないとき、それはメモリリークを避けるために、少し簡単だと私は個人的には、最初のオプションを提案。

Answer 2

割り当てられたスペースをすべて解放する必要がありますが、プログラムの他の部分でこれらのメモリスペースを使用するため、そのスペースを返す場合は、使用した後に解放する必要があります。関数を呼び出すコード内のすべての場所を参照し、戻り値を使用した後に領域を解放します。

Answer 3

ブロックのアドレスを返す予定がある場合は、ブロックを解放する必要はありませんが、代わりに呼び出しコードを使用して後で解放してください（）。これはオンウェルジパスと呼ばれています。

関数内で解放してポインタを返すと、呼び出したコードは、すでに解放されているブロックにアクセスしようとして、未定義の動作になります。

Answer 4

これで終了します。 malloc（）と一致するfree（）が同じ関数内になければならないという規則はありません。

Answer 5

これはいくつかの既存の機能（例：strdup()）の習慣ですが、一般的には悪い考えです。関数呼び出しの中で何が起こるのかをユーザに認識させることは、必要条件ではありません。内部関数がどれだけ隠されているのかを考えてください。 一般的に言えば、ユーザーにバッファーとメモリーを割り当てる代わりにサイズを渡したい場合があります。あなたは `のchar *`を格納するのに十分なメモリを割り当てるが、char` `を指すポインタに代入（とリターンをキャストする必要はありませんしている - `のmalloc（） `ほぼ確実に間違っていると