2014-01-05 77 views
1

所以基本上我的输入是一个未知大小的字符串。 我所做的是创建一个函数来扫描它并为每个字符动态分配内存。动态字符串内存泄漏

char* GetUnknownStr(){ 
char* string = NULL; 
char input = 0; 
int charCount = 0; 
//scan for chars until ENTER is pressed 
while (input != '\n'){ 
    scanf("%c", &input); 
    //on last iteration (when ENTER is pressed) break from loop 
    if (input == '\n') 
     break; 
    //realloc new amount of chars 
    string = (char*)realloc(string, (++charCount)*sizeof(char)); 
    //check if realloc didnt fail 
    if (string == NULL){ 
     puts("failed to allocate"); 
     return 0; 
    } 
    //assign char scanned into string 
    string[charCount-1] = input; 
} 
//realloc 1 more char for '\0' in the end of string 
string = (char*)realloc(string, (++charCount)*sizeof(char)); <--leak 
string[charCount-1] = '\0'; 
return string; 
} 

的valgrind:

==30911== HEAP SUMMARY: 
==30911==  in use at exit: 4 bytes in 1 blocks 
==30911== total heap usage: 7 allocs, 6 frees, 16 bytes allocated 
==30911== 
==30911== 4 bytes in 1 blocks are definitely lost in loss record 1 of 1 
==30911== at 0x4A05255: realloc (vg_replace_malloc.c:476) 
==30911== by 0x40091F: GetUnknownStr (in /u/stud/krukfel/C/e5/a.out) 
==30911== by 0x40266B: main (in /u/stud/krukfel/C/e5/a.out) 
==30911== 
==30911== LEAK SUMMARY: 
==30911== definitely lost: 4 bytes in 1 blocks 
==30911== indirectly lost: 0 bytes in 0 blocks 
==30911==  possibly lost: 0 bytes in 0 blocks 
==30911== still reachable: 0 bytes in 0 blocks 
==30911==   suppressed: 0 bytes in 0 blocks 
==30911== 
==30911== For counts of detected and suppressed errors, rerun with: -v 
==30911== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 6 from 6) 

由于某种原因,即时得到了内存泄漏,不能找出原因,使用Valgrind的尝试,但不可能从它什么太大的意义。 谢谢!

+1

Valgrind告诉你什么? –

+0

你怎么知道你第一次有泄漏? – Devolus

回答

2

除了小的文体问题,如铸造malloc和乘以sizeof(char),你的代码是好的。实际的泄漏是在调用你的函数的代码中。

处理内存泄漏的中心问题是拥有权。一旦知道谁拥有分配的内存块,就知道需要释放哪些内容以避免泄漏。在这种情况下,该字符串在GetUnknownStr内部创建,但是该字符串的所有权将转移给调用者。它是您的GetUnknownStr函数的调用者,负责调用free函数返回的结果。如果主叫方未释放该字符串,则会报告泄漏。泄漏的位置将是最后一次重新分配的位置,即您在代码中标记的行。

+0

好吧,基本上发生的是,我用它来扫描一个字符串(主),然后立即释放它。 –

+0

@FelixKreuk它看起来像调用者(即你的'main')跳过这些调用中的一个来释放字符串。如果这是'realloc'的唯一函数,那么看起来您已经调用了它七次。用户最后一次输入三个字母(可能是“再见”命令?),此时'main'退出而不调用'free'。 – dasblinkenlight

+0

谢谢!就是这样! –