Question

我想知道为什么valgrind说：

    ==9952== 30 bytes in 6 blocks are definitely lost in loss record 1 of 1
    ==9952==    at 0x4C2BF0E: realloc (vg_replace_malloc.c:662)
    ==9952==    by 0x40131F: setCharsPositions (paramsExec.c:99)
    ==9952==    by 0x400CF3: main (main.c:87)

我无法弄清楚我realloc()的问题是什么（你不必说每次重新分配一个内存是低效的......）;变量char **passwordSet2是全局的 - 也许这就是问题......如果我做错了什么，请告诉我！我疯了！

void setCharsPositions(char *charsPos){

    int i, k;
    char *posStr = NULL;

    for(i = 0; i < strlen(charsPos); i++){

        posStr = malloc(sizeof(char));

        if(charsPos[i] == '['){

            for(k = 0, i++; charsPos[i] != ','; i++, k++){
                posStr[k] = charsPos[i];
                posStr = realloc(posStr, (k+2)*sizeof(char));
            }
            posStr[k] = '\0';

            passwordSet2[atoi(posStr)-1] = malloc(sizeof(char));
            for(k = 0, i++; charsPos[i] != ']'; i++, k++){
                passwordSet2[atoi(posStr)-1][k] = charsPos[i];
                passwordSet2[atoi(posStr)-1] = realloc(passwordSet2[atoi(posStr)-1], (k+2)*sizeof(char));
            }
            passwordSet2[atoi(posStr)-1][k] = '\0';
        }
        free(posStr);
    }
}

Answer 1

passwordSet2[atoi(posStr)-1] = malloc(sizeof(char));

如果先前已分配passwordSet2[atoi(posStr)-1]，则会泄漏内存。也就是说，如果posStr的相同值出现多次，则会发生泄漏。丢失的内存将来自之前的realloc，这就是valgrind将报告的内容。（如果你指出哪一行是第662行会有所帮助。）如果posStr的相同值不止一次发生但不应该发生，那么你需要检查一下。如果允许多次出现，则应添加对free的调用：

free(passwordSet2[atoi(posStr)-1]);
passwordSet2[atoi(posStr)-1] = malloc(sizeof(char));

当free为NULL时，passwordSet2[atoi(posStr)-1]的第一次出现是无操作。

此外，您没有检查malloc和realloc的NULL返回，但这些会导致程序崩溃，而不仅仅是泄漏。

最后，为了清晰和可能的效率，我建议你把atoi（posStr）-1放入变量而不是重复它。

全局变量和realloc以及内存泄漏

1 个答案: