1

因此，我正在尝试将 .s19 文件中的 s 记录加载到内存中，以用于我正在处理的任务及其工作。但是，当我从我的代码中删除一个未使用的数组时，一切都会停止工作并崩溃。

未使用的数组是：

char test[65536];

这是我写的加载器：

void loader(FILE * srec)
{
    char instring[SREC_LEN];
    char test[65536]; // This isn't used, but the program crashes without it for some reason
    int i=0;
    int j=0, k,l;
    while (fgets(instring, SREC_LEN, srec) != NULL)
    {

        while(instring[i] != '\n') // Counts the characters in the s-record
        {
            i++;

        }
        j = j+i;
        for(k=0;k<=i;k++) // Puts the records into memory
        {
            memory[l] = instring[k];
            l++;
        }
        l = j;

    }
    #ifdef DEBUG
    printf("MEMORY: %s",memory);
    #endif // DEBUG
}

如果您能帮助我理解为什么会发生这种情况，我将不胜感激。

4

1 回答 1

1

你的代码有未定义的行为，它只能靠运气：

fgets()EOF如果过早到达，可能会在不将换行符写入缓冲区的情况下返回。所以你至少应该在你的循环中考虑到这一点。你也永远不会重置i为 0，你应该这样做。改变这个：

    while(instring[i] != '\n') // Counts the characters in the s-record
    {
        i++;

    }

至：

    i = 0;
    while(instring[i] != '\n' && instring[i] != '\0') // Counts the characters in the s-record
    {
        i++;

    }

l从未初始化；您可能在memory. 初始化l为 0：

int j = 0, k, l = 0;

（我假设它memory足够大，可以容纳所有东西）。

在我看来，它也像您想要的那样，for(k = 0; k < i; k++)而不是for(k = 0; k <= i; k++)，因为i是您要复制的字符数。

您可能想memcpy()改用。

于 2015-07-04T22:31:04.327 回答