时间戳

首页 > 解决方案 > 涉及未初始化字符串的 Valgrind 错误：错误标志？

valgrind - 涉及未初始化字符串的 Valgrind 错误：错误标志？

问题描述

当运行 valgrind 以检查用 C89/90 编写的程序中的错误时，它会Uninitialised value was created by a heap allocation为我编写的函数出现错误strToUpper()，尽管字符串已初始化。

我正在使用这个函数来比较字符串忽略大小写。不幸的是，C89 中似乎没有包含该strcasecmp()函数<string.h>，所以我编写了自己的函数，它调用了strToUpper()andstrcmp()函数。

代码

char* strToUpper(char* inStr)
{
    int i;
    char *upperStr;
    size_t strLen = strlen(inStr);
    upperStr = (char*)malloc(sizeof(char) * (strLen + 1));

    /* Does this for loop not initialise upperStr? */
    for (i = 0; i < strLen; i++)
        upperStr[i] = toupper(inStr[i]);

    return upperStr;
}

瓦尔格林错误

==27== Conditional jump or move depends on uninitialised value(s)
==27==    at 0x4C31FAA: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==27==    by 0x406649: strcasecmp (stringPlus.c:178)
==27==    ...
==27==    by 0x400FEB: main (main.c:58)
==27==  Uninitialised value was created by a heap allocation
==27==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==27==    by 0x4062E4: strToUpper (stringPlus.c:58)
==27==    by 0x406622: strcasecmp (stringPlus.c:175)
==27==    ...
==27==    by 0x400FEB: main (main.c:58)

有任何想法吗？

标签： valgrindc89

解决方案

您没有终止复制的字符串。

添加类似的东西

upperStr[i] = '\0';

在你的 for 循环之后。

推荐阅读