調用 malloc() 后 Free() 無法按預期工作

Question

我用 C 編寫了以下函數，但有兩個問題：

1. 我被要求釋放動態分配的內存以防止使用過多的 RAM，但free(word)只會導致我的程序出現錯誤。 如果我刪除它一切正常，為什么會發生這種情況？ 我按照許多不同文章中的建議使用完word后免費使用。

2. 我被要求使用具有最小所需空間的 malloc，但我該怎么做？ 目前我的代碼分配max_str_len的RAM塊，但如果字要短得多像a字母，我不希望分配200塊了點。 請問有什么建議嗎？

int read_words(char *words[], int size, int max_str_len) {
    char *word;
    char ch;
    int i;
    for (i = 0; i < size; ++i) {
        word = (char *)malloc((max_str_len + 1) * sizeof(char));
        if (word == NULL)
            return -1;
        for (int j = 0; j < max_str_len; ++j) {
            scanf("%c", &ch);
            if (ch == '\n') break;
            if (ch == EOF || ch == 'R') return i;
            word[j] = ch;
        }
        words[i] = word;
        free(word);
    }
    return i;
}

Answer 1

您將malloc 'd char*放入調用者提供的words[i] ，然后釋放它。 那沒有意義。 如果你釋放它，調用者就不能對它做任何事情。

如果你想要malloc 'd串是最小的，你可以realloc他們或者你可以讀入一個大的緩沖區（在函數的開始也許分配），然后將結果復制到malloc倒是緩沖區的大小恰到好處。

請注意，您也未能檢查scanf是否存在錯誤，並且如果您在函數中間出現內存故障，則會導致內存泄漏——通過返回 -1，您實際上會丟失有關已填充words元素數量的信息擁有指針。 您可能希望返回該信息（ return i; ）或在malloc失敗之前釋放該函數分配的所有指針。

Answer 2

您的代碼中有多個問題：

• 您釋放了為每個字分配的內存，但您返回指向調用者提供的數組中已釋放塊的指針，當調用者取消引用這些指針時會導致未定義的行為。
• 您對文件結尾的測試不正確：然后scanf()將返回EOF ，但該字符不會被設置為EOF ，這可能不適合char 。 您應該使用getchar()並使ch成為int 。
• 您應該在讀取的字符串末尾設置一個空終止符。
• 一旦知道字符串長度，就可以使用realloc來縮小內存塊。

這是一個修改后的版本：

int read_words(char *words[], int size, int max_str_len) {
    char *word, *p;
    int i, j, ch;
    for (i = 0; i < size;) {
        word = (char *)malloc(max_str_len + 1);
        if (word == NULL) {
            /* free the words allocated so far and return a failure code */
            while (i-- > 0)
                free(words[i];
            return -1;
        }
        for (j = 0; j < max_str_len; ++j) {
            ch = getchar();
            if (ch == '\n' || ch == EOF || ch == 'R') break;
            word[j] = ch;
        }
        if (j == 0 && (ch == EOF || ch == 'R'))
            break;
        word[j] = '\0';
        p = (char *)realloc(word, j + 1);
        if (p != NULL)
            word = p;
        words[i++] = word;
        if (ch == EOF || ch == 'R')
            break;
    }
    return i;
}