我正在用C语写一个“谁想要成为百万富翁”游戏,其中包括SDL。 我已经开发了SDL图形和单独的数据处理。 后者就是那个,我遇到了问题。
它获得了大约5000行长的.csv文件,并在动态分配的内存的帮助下将细节放入结构中。 然后它被打印到控制台。 但是,它只能每隔3次左右工作一次。 其他时候,该程序冻结。 试过调试,它说:
编程接收信号SIGTRAP,跟踪/断点陷阱。
我想把问题缩小到这个部分:
while ((read = getline(&line, &n, kerdes)) != -1) {
sp = p = line;
field = 0;
// The below line triggers the signal
questions[cnt] = (Question*) malloc(sizeof(Question));
// Cuts
while (*p != '\0') {
if (*p == ',') {
*p = 0;
if (field == 0) questions[cnt]->nth = atoi(sp);
if (field == 1) questions[cnt]->question_to = strdup(sp);
if (field == 2) questions[cnt]->answer_a = strdup(sp);
if (field == 3) questions[cnt]->answer_b = strdup(sp);
if (field == 4) questions[cnt]->answer_c = strdup(sp);
if (field == 5) questions[cnt]->answer_d = strdup(sp);
if (field == 6) questions[cnt]->answer_r = strdup(sp);
if (field == 7) questions[cnt]->cat = strdup(sp);
*p = ',';
sp = p + 1;
field++;
}
p++;
}
cnt++;
}
getline
函数是来自this answer的函数:
size_t getline(char **lineptr, size_t *n, FILE *stream) {
char *bufptr = NULL;
char *p = bufptr;
size_t size;
int c;
if (lineptr == NULL) {
return -1;
}
if (stream == NULL) {
return -1;
}
if (n == NULL) {
return -1;
}
bufptr = *lineptr;
size = *n;
c = fgetc(stream);
if (c == EOF) {
return -1;
}
if (bufptr == NULL) {
bufptr = malloc(128);
if (bufptr == NULL) {
return -1;
}
size = 128;
}
p = bufptr;
while(c != EOF) {
if ((p - bufptr) > (size - 1)) {
size = size + 128;
bufptr = realloc(bufptr, size);
if (bufptr == NULL) {
return -1;
}
}
*p++ = c;
if (c == '\n') {
break;
}
c = fgetc(stream);
}
*p++ = '\0';
*lineptr = bufptr;
*n = size;
return p - bufptr - 1;
}
我做了 - 希望 - 搜索stackoverflow足够彻底,没有任何成功。
可能导致问题的原因是什么? 在我看来,过度索引并不在其后面,并且free()使用得很好。
请在以下链接中找到pastebin上的整个.c文件: Click here
可以使用以下链接访问CSV文件(非英语):Click here
答案 0 :(得分:0)
问题出现在getline
代码中,非常简单。我用 POSIX 标准getline
替换它,它完美无缺。问题是这里的realloc
代码:
p = bufptr;
while(c != EOF) {
if ((p - bufptr) > (size - 1)) {
size = size + 128;
bufptr = realloc(bufptr, size);
if (bufptr == NULL) {
return -1;
}
}
*p++ = c;
if (c == '\n') {
break;
}
c = fgetc(stream);
}
realloc
可以(并且很可能将)返回指向 new 缓冲区的指针,但p
将指向旧的缓冲区,因此行*p++ = c;
将具有未定义的行为。