realloc을() I 문자열에 문자를 추가하는 기능을 가지고 메모리

Question

누설 :

void AddChToString(char **str,char ch){ 
    int len=(*str)?strlen(*str):0; 
    (*str)=realloc(*str, len+2); 
    (*str)[len]=ch; 
    (*str)[len+1]='\0'; 
} 

하고 (Mac) 인스트루먼트 Valgrind의를 표시하는 선이 (* STR) = realloc을 (* STR , len + 2)에 메모리 누수가 있습니다. 이 realloc 구현 문제가 있습니까? 아니면 부적절하게 사용하고 있습니까?

==39230== 6 bytes in 1 blocks are definitely lost in loss record 1 of 7 
==39230== at 0x100018B2D: realloc (vg_replace_malloc.c:525) 
==39230== by 0x100002259: AddChToString (in ./OpenOtter) 
==39230== by 0x10000477B: QueryMapFromString (in ./OpenOtter) 
==39230== by 0x100684CD2: ??? 
==39230== by 0x100001FB0: RequestHandler (in ./OpenOtter) 
==39230== by 0x100065535: _pthread_start (in /usr/lib/libSystem.B.dylib) 
==39230== by 0x1000653E8: thread_start (in /usr/lib/libSystem.B.dylib) 
==39230== 
==39230== 9 bytes in 1 blocks are definitely lost in loss record 2 of 7 
==39230== at 0x100018B2D: realloc (vg_replace_malloc.c:525) 
==39230== by 0x100002259: AddChToString (in ./OpenOtter) 
==39230== by 0x10000298E: ParseHTTPRequest (in ./OpenOtter) 
==39230== by 0x100004151: OpenRoutesFile (in ./OpenOtter) 
==39230== by 0x10000142B: main (in ./OpenOtter) 
==39230== 
==39230== 45 bytes in 5 blocks are definitely lost in loss record 3 of 7 
==39230== at 0x100018B2D: realloc (vg_replace_malloc.c:525) 
==39230== by 0x100002259: AddChToString (in ./OpenOtter) 
==39230== by 0x10000298E: ParseHTTPRequest (in ./OpenOtter) 
==39230== by 0x100001EB4: RequestHandler (in ./OpenOtter) 
==39230== by 0x100065535: _pthread_start (in /usr/lib/libSystem.B.dylib) 
==39230== by 0x1000653E8: thread_start (in /usr/lib/libSystem.B.dylib) 
==39230== 
==39230== LEAK SUMMARY: 
==39230== definitely lost: 60 bytes in 7 blocks 
==39230== indirectly lost: 0 bytes in 0 blocks 
==39230==  possibly lost: 0 bytes in 0 blocks 
==39230== still reachable: 1,440 bytes in 4 blocks 
==39230==   suppressed: 0 bytes in 0 blocks 

감사 : 여기서

는 Valgrind의에서 출력된다.

Answer 1

realloc 자체에 대한 호출은 메모리 누출이 아닙니다. 재 할당 된 문자열에 대한 메모리가 더 이상 필요없는 후에는 무료로 릴리스되는지 확인해야합니다.

Answer 2

계측기가 실제 누출이 있거나 누출 가능성이 있음을 나타 냅니까?

을 사용하면 이 누출됩니다.realloc()이 실패하면 메모리가 누출됩니다. 이 경우 NULL이 반환되지만 원본 블록은 해제되지 않습니다. 그래서 당신은 블록에 대한 포인터를 잃어버린다. 포인터가 다른 곳에 저장되어 있지 않다면 그것을 해제 할 수 없다.

그러나 이것은 드문 경우입니다 (예 : 메모리를 모두 소모했을 때).

이 당신의 도구에 대해 불평하는 무엇, 당신 같은 뭔가 누출 경고를 해결 할 수 있어야한다 :

void AddChToString(char **str,char ch){ 
    int len=(*str)?strlen(*str):0; 
    char* tmp = realloc(*str, len+2); 

    if (!tmp) { 
     // whatever error handling is appropriate 
    } 

    (*str)=tmp; 
    (*str)[len]=ch; 
    (*str)[len+1]='\0'; 
} 

Answer 3

시도 그 다음에 STR 변수를 얻기 위해 strcpy와 전화, 별도의 변수를 사용하여 재 할당 공백은 다음과 같습니다.

void AddChToString(char **str,char ch){ 
char *blah; 
int len=(*str)?strlen(*str):0; 
blah=realloc(NULL, len+2); 
strcpy(blah, str); 
(*str)[len]=ch; 
(*str)[len+1]='\0'; } 

Answer 4

내가 realloc에 대한 구현 문제에 대해 잘 모르겠지만,이 코드에서 메모리 누수의 기회 반드시이 다음 realloc 맨 페이지에서 :

하면 realloc()는 원래 블럭은 건드리지 않고 남아있다 실패 ; 그것은 해방되거나 움직이지 않는다.

하고, realloc 이후 반환 NULL를 실패, 당신이 메모리 누수가 있도록, 이미 할당 된 메모리 블록에 유일한 포인터를 잃고 실패하는 경우.

문제를 방지하려면, 당신은 수행해야합니다

char * temp=realloc(*str, len+2); 
if(temp==NULL) 
{ 
    /* handle the error in some way */ 
} 
else 
    *str=temp;