来自 valgrind 的未初始化值

Question

==2630== Conditional jump or move depends on uninitialised value(s)
==2630==    at 0x4E82D71: vfprintf (in /usr/lib64/libc-2.21.so)
==2630==    by 0x4E88E78: printf (in /usr/lib64/libc-2.21.so)
==2630==    by 0x400C0C: searchWord (T9.c:91)
==2630==    by 0x400A0A: main (T9.c:40) 

==2114==  Uninitialised value was created by a heap allocation
==2114==    at 0x4C28C50: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==2114==    by 0x400FD1: newStr (trie_node.c:125)
==2114==    by 0x400F8C: create_trie (trie_node.c:117)
==2114==    by 0x4009D5: main (T9.c:37)

我在 valgrind 中通过运行跟踪函数收到上面的错误消息。我很确定我已经初始化了变量。这是结构代码：

struct wordList* newStr(char* text) {
    char* word;
    struct wordList* tmp = (struct wordList*)malloc(sizeof(struct wordList));
    word = (char *)malloc(sizeof(char) * strlen(text) + 1);
    strncpy(word, text, strlen(text));
    tmp->str = word;
    tmp->next = NULL;
    return tmp;
}

第 91 行 T9.c 周围的代码：

struct wordList* cur;
if (cur && invalid == 0 && flag == 0) {
  printf("\t\'%s\'\n", cur->str);
 }

更新：

我修改了来自

的strncpy行

    strncpy(word, text, strlen(text));

到

word = strncpy(word, text, strlen(text));

这解决了未初始化的问题，但是我收到了我不理解的新错误消息：

==3245== Invalid read of size 1
==3245==    at 0x4E82D71: vfprintf (in /usr/lib64/libc-2.21.so)
==3245==    by 0x4E88E78: printf (in /usr/lib64/libc-2.21.so)
==3245==    by 0x400C0C: searchWord (T9.c:91)
==3245==    by 0x400A0A: main (T9.c:40)
==3245==  Address 0x51f7d45 is 0 bytes after a block of size 5 alloc'd
==3245==    at 0x4C28C50: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==3245==    by 0x400FC1: newStr (trie_node.c:124)
==3245==    by 0x400F80: create_trie (trie_node.c:117)
==3245==    by 0x4009D5: main (T9.c:37)

Answer 1

我需要查看更多代码，但我不确定这是否正是 valgrind 所抱怨的，您的代码确实存在重大错误：

strncpy(word, text, strlen(text));

您没有以 null 终止您的字符串，您只是在复制实际字符。这特别有趣，因为有一个函数已经负责分配正确的内存量和复制字符串：strdup.

还必须警告停止转换 malloc 的 return 值。

Answer 2

您必须查看完整的错误（而不仅仅是第二个）。根据经验，您应该从上到下阅读错误，因为后续错误可能是前者（或前者的解释）的结果。第二条消息只是告诉您 malloc 分配的内存未初始化，这是预期的 - 它只是对第一条消息的修正。

错误在于当您使用分配的 space 时没有填充有用的东西。错误在于新发布的代码，cur->str 指向的块可能未初始化。

故障有点棘手，因为你使用strncpy。它只会复制 strlen(text) 字节，从而跳过空终止符。你已经为它创建了 space 但它没有被复制，因此分配缓冲区的最后一个字节没有被初始化（错误发生在 vfprintf 遍历字符串并到达空终止符应该的字节时是）。

明确strncpy(dst, src, cnt)的作用。它最多复制 cnt 字节（包括空终止符），如果包含空终止符的 src 字符串不超过 cnt 字节，它将复制完整的字符串。否则它只会复制字符串的前 cnt 个字节并且 not null 终止 dst.

来自 valgrind 的未初始化值

Uninitialized value from valgrind

c

memory

struct

valgrind