c：割り当てられたオブジェクトのエラーを解放する

Question

さて、私は呼び出すメソッドを持っています。呼び出されたメソッドは、呼び出されたメソッドの内部に割り当てられたchar*を返します。返されたchar*を新しいchar*に割り当てた後、私はそれを使用した後で解放したい。しかしXcodeは文句を言ってエラーを投げます。それは私が自由にしようとするオブジェクトは割り振られていないと言いますが、私はそれを割り当てると確信しています。ここで

は私のコードです：

void print_r() 
{ 

    char *stateA = isActive(ruter_array[id]->flagg); 

    //print out the results etc 

    free(stateA);  <-----The program crashes here. 


} 

char * isActive(unsigned char a_flag) 
{ 
    char *ret = malloc(5); 

    if((a_flag & 1) == 1) 
    { 
     //is active 
     ret = "yes"; 
     return ret; 
    } 
    ret = "no"; 
    return ret; 

} 

これが何をするのか理解することは重要ではない。その、なぜこの解放がクラッシュしていますか？

Answer 1

ret = "no"; 

ポインタを上書きします。変更：

strcpy(ret, "no"); 

"yes"の場合も同様です。

@DavidSchwartzによって指摘されているように、あなたの目的のために、おそらく動的に割り当てられたメモリを使用する必要はありません。ただ、直接文字列リテラルを返すことができます：その場合の

char * isActive(unsigned char a_flag) 
{ 
    if((a_flag & 1) == 1) 
    { 
     //is active 
     return "yes"; 
    } 
    return "no"; 
} 

そしてもちろんの呼び出し元すべきではないfree返された値を。

Answer 2

メモリを割り当て、そのメモリの最初のアドレスをポインタretに割り当てますが、"no"と"yes"はコンパイル時にメモリに静的に格納される文字列リテラルです。 ret = "no"を実行すると、ポインターが文字列リテラル"no"の最初の文字を指すように変更されます。これで、割り当てたメモリにアクセスできなくなり、メモリリークが発生しました。しかし、クラッシュは、malloc()またはその友人によって割り当てられていないメモリを解放しようとしているためです。

Answer 3

あなたもstrdupを使用することができます書かれhereとして

char * isActive(unsigned char a_flag) 
{ 
    return strdup((a_flag & 1) == 1 ? "yes" : "no"); 
} 

それも解放できるようにポインタがまだmalloc経由で割り当てられました。