4

我正在尝试将字符串拆分为句子（由句子分隔符分隔）。代码本身可以工作，但我在函数中不断出现内存泄漏。

char ** splitSentences(char *string) {

int sentencecount = 0;
char* buf = NULL;
char* str = NULL;

buf = malloc((strlen(string) + 1) * sizeof(char));
strcpy(buf,string);

str = buf;

sentencecount = countSentences(str);

if(sentencecount != 0)
{
    char** sentences = NULL;
    sentences = malloc((sentencecount + 1)*sizeof(char*));
    memset(sentences,0,sentencecount+1);

    char* strToken = NULL;
    strToken = malloc((strlen(str)+1)*sizeof(char));
    memset(strToken,0,strlen(str)+1);

    strToken = strtok(str, SENTENCE_DELIMITERS);

    int i = 0;

    while(strToken != NULL) {
        sentences[i] = NULL;
        sentences[i] = malloc((strlen(strToken)+1)*sizeof(char));
        strncpy(sentences[i], strToken,strlen(strToken) + 1);
        strToken = strtok(NULL, SENTENCE_DELIMITERS);
        i++;
    }

    sentences[sentencecount] = NULL;

    //Free the memory
    free(strToken);
    strToken = NULL;

    free(buf);
    buf = NULL;

    return sentences;
}

return NULL;

}

我找不到它为什么会泄漏内存。有人知道吗？

4

4 回答 4

9

这是内存泄漏：

strToken = malloc((strlen(str)+1)*sizeof(char));
// ...
strToken = strtok(str, SENTENCE_DELIMITERS);

您使用 为对象分配空间malloc，然后在调用 后丢失指向该空间的指针strtok。

于 2010-08-05T15:08:16.440 回答
1

你的malloc句子和return它给来电者。你在那里免费吗？

于 2010-08-05T15:08:33.723 回答
1

strtok()返回指向在字符串中找到的标记的指针。在您的示例中，我认为您不需要分配 strToken 变量（它只是一个指针）。尝试删除：

strToken = malloc((strlen(str)+1)*sizeof(char));
memset(strToken,0,strlen(str)+1);
于 2010-08-05T15:22:44.410 回答
0

您不应该 malloc 用于保存 strtok 的返回值的字符串。检查strtok的参考。因此，memleak。

于 2010-08-05T15:52:50.803 回答