関数の外部の空きメモリ

Question

これは正しいですか、それはCのメモリをリークさせるでしょうか？

unsigned char * prep(int length,int args, ...) 
{ 
    unsigned char *message = (unsigned char *) malloc(length); 

    va_list listp; 
    va_start(listp, args); 

    int i = 0; 
    int len = 0; 
    unsigned char *source_message ; 
    int step = 0; 
    for(i = 0 ; i < args; i++) 
    { 

    source_message = va_arg(listp, unsigned char *); 
    len = va_arg(listp, long); 
    memcpy(message+step, source_message, (long) len); 
    step+=len; 

    } 
    va_end(listp); 
return message; 
} 

それを呼び出すと、正しいです

unsigned char *mess = prepare_packet_to_send(some vars here); 
free(*mess); 

Answer 1

C関数strdup（）は同じことをしますが、使用すると解放するための規約に従わなければならない場合もあります。しかし、あなたのコードでは大きな問題があることが分かりました。実際に割り当てたサイズより多くのデータを書き込むことができます。 これを修正する試みです。

unsigned char * prep(int length,int args, ...) 
{ 
int i = 0; 
int len = 0; 
unsigned char *source_message ; 
int step = 0; 
unsigned char *message = (unsigned char *) malloc(length); 
va_list listp; 
va_start(listp, args); 
for(i = 0 ; i < args && step <= length; i++) { 
    source_message = va_arg(listp, unsigned char *); 
    len = va_arg(listp, long); 
    memcpy(message+step,source_message,(step+len >length)?length-step:len); 
    step+=len; 
} 
va_end(listp); 
return message; 
} 

のように使用してください。

char * p = prep(size,2,"message 1",9,"message 2",9); 
if (p) { 
.... work with p.... 
    free(p); 
} 

Answer 2

外のポインタを解放よりも、それはメモリリークが発生します。その関数の戻り値を解放することを忘れない限り、あなたはうまくいくはずです。

Answer 3

これは完全に合法です。関数は、他の場所で解放される予定のメモリを返すことがあります。実際、使用するmalloc関数は、この正確な契約上の振る舞いを持っています。

他の問題はないことを確認するためにコードを慎重に踏んだわけではありませんが、mallocされたメモリを返すことは間違いありません。

Answer 4

あなたはこのように、独自の機能を使用します。

unsigned char *p = prep(100, 1, "hello", 3); 
// ... 
free(p); 

、その後、あなたは何のメモリリークを持っていないでしょう。