c - 删除未使用的变量会导致代码崩溃

Question

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

未使用的数组是:

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
}

如果您能帮助我理解为什么会这样，我将不胜感激。

Answer 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()相反。