所以,valgrind 给了我那个错误,我用 --track-origins=yes 运行它并找到了错误所在的行,但我不明白错误是什么或如何修复它。
#include <stdio.h>
#include <stdlib.h>
typedef struct Date{
int year;
int month;
int day;
} Date;
typedef struct Data{
Date date;
float temp;
float uncertainty;
char country[100];
} Data;
int main(){
FILE* f = fopen("tempcountries_short.csv", "r");
char* line = NULL;
int capacity = 0;
int countries_capacity = 0;
int line_ix = 0;
char c;
Data* country = NULL;
while ((c = fgetc(f)) != '\n'){
if (line_ix + 1 > capacity){
if (capacity == 0)
capacity = 10;
else
capacity = capacity * 2;
line = realloc(line, capacity);
}
line[line_ix] = c;
line_ix++;
}
if (countries_capacity == 0)
countries_capacity = 10;
else
countries_capacity = countries_capacity * 2;
country = realloc(country, countries_capacity);
printf("%i\n",sscanf(line, "%i - %i - %i, %f , %f , %s",
&country->date.year, &country->date.month,
&country->date.day, &country->temp, &country->uncertainty,
country->country));
}
这是 Valgrind 的输出,有选项 --leak-check=full 和 --track-origins=yes:https://pastebin.com/EyqDGBmQ正如你所看到的,还有很多其他错误,我不明白是什么导致了它们。
该程序正在从包含有关许多国家/地区温度的数据的文件中读取行,我只是采用了单个国家/地区的代码部分来复制错误,但 country 应该是许多数据结构的数组。这是我正在阅读的文件中的示例行:
1972-03-01,4.787,0.342,斯洛伐克
country = realloc(country, countries_capacity);
你似乎依赖于realloc(0, new_size)
表现得像malloc(new_size)
。这很好,但是您必须确保传递给的指针变量realloc
确实为空。在此之前没有代码初始化country
变量,并且在它的声明中......
Data* country;
...没有初始化程序。将此更改为
Data *country = 0;
那部分问题应该消失。
通常,当您从 中得到一整串错误时valgrind
,只有第一个是有意义的,所以看看这是否能解决您的所有问题。
编辑:使用上述更正的程序版本,输入显示的输入示例行,我没有收到关于内部未初始化值的任何投诉realloc
,但我仍然收到关于内部未初始化值的投诉sscanf
:
==28542== Conditional jump or move depends on uninitialised value(s)
==28542== at 0x4C340E6: rawmemchr (vg_replace_strmem.c:1409)
==28542== by 0x4EB6291: _IO_str_init_static_internal (strops.c:41)
==28542== by 0x4EA476C: __isoc99_vsscanf (isoc99_vsscanf.c:41)
==28542== by 0x4EA46D3: __isoc99_sscanf (isoc99_sscanf.c:31)
==28542== by 0x108886: main (in /tmp/a.out)
和关于无效写入的投诉:
==28542== Invalid write of size 4
==28542== at 0x4E98FFB: _IO_vfscanf (vfscanf.c:1898)
==28542== by 0x4EA4781: __isoc99_vsscanf (isoc99_vsscanf.c:43)
==28542== by 0x4EA46D3: __isoc99_sscanf (isoc99_sscanf.c:31)
==28542== by 0x108886: main (in /tmp/a.out)
==28542== Address 0x51f41a8 is 8 bytes inside a block of size 10 alloc'd
==28542== at 0x4C2CABF: malloc (vg_replace_malloc.c:298)
==28542== by 0x4C2EE04: realloc (vg_replace_malloc.c:785)
==28542== by 0x10883C: main (in /tmp/a.out)
这些都是由真正的错误引起的:第一个是因为line
数组从未被制作成正确的 C 字符串(通过添加 nul 终止符),第二个是因为传递给的大小realloc
是错误的。你需要
line[line_ix] = '\0';
紧接在while
循环之后,以及
country = realloc(country, countries_capacity * sizeof(Data));
而不是realloc
你现在的电话。
通过这些更改,我不会收到 valgrind 的投诉。你仍然有你正在使用的问题sscanf
,这总是一个坏主意,但 valgrind 无法帮助你。
本文收集自互联网,转载请注明来源。
如有侵权,请联系 [email protected] 删除。
我来说两句