堆栈变量已损坏

时间:2013-07-28 20:38:58

标签: c stack hamming-code stack-corruption

它因调试错误而崩溃,并且表示变量'code'周围的堆栈已损坏。这是我正在做的汉明码实验室的代码。输入文件只是同一行上的一堆1和0。为什么会崩溃?

void processFile(FILE* read, char* InMessage) {
    int i = 0, count = 0;

    for (i = 0; !feof(read); i++) {
            InMessage[i] = fgetc(read);
        count++;
    }

    InMessage[count] = '\0';
}

void hammingCode(char* InMessage) {
    int len = strlen(InMessage), i = 0, j = 0;
    char code[12], temp[1000];

    temp[0] = '\0';

    for (i = 0, j = 0; i < len; i++, j++) {
        code[j] = InMessage[i];
        if (j == 10) {
            j = 0;

            decode(code);
            code[11] = '\0';
            strcat_s(temp, sizeof(char)*1000, code);
        }
    }

    strcpy_s(InMessage, sizeof(char)*1000, temp);
}

void decode(char* codeWord) {
    int i = 0, j = 0, parity[4] = {0}, diffParity[4] = {0}, twoPower = 0, readNSkip =     0, bitSum = 0;

    for (i = 0; i < 4; i++) {
        twoPower = (int)pow((double)2, i);

        for (j = twoPower; j <= 12; j++) {
            if (readNSkip <= twoPower) {
                if (j != twoPower)  {
                    parity[i] += codeWord[j-2] - 48;
                }
                readNSkip++;
            }
            else {
                if (readNSkip == twoPower*2)
                    readNSkip = 0;
                readNSkip++;
            }
        }

        if (parity[i] % 2 == 0)
            parity[i] = 0;
        else
            parity[i] = 1;

        if ((codeWord[twoPower-1] - 48) != parity[i])
            diffParity[i] = 1;
    }

    for (i = 0; i < 4; i++) {
        twoPower = (int)pow((double)2, i);
        bitSum += diffParity[i]*twoPower;
    }

    codeWord[bitSum] = !codeWord[bitSum];

}

1 个答案:

答案 0 :(得分:1)

我在这里看到两个问题:

  1. 我认为您在InMessage函数中错误地计算了hammingCode缓冲区的大小:

    int len = strlen(InMessage), i = 0, j = 0;
    

    strlen函数通过查找第一个空终止符的位置来确定字符串的长度。如果InMessage没有被清除,那么这可能会给你一些奇怪的长度,因为它将包含一个随机的字节序列。相反,如果您已清除缓冲区,则len将为0。

    要解决此问题,调用者最好提供缓冲区的大小:

    int hammingCode (char *InMessage, size_t messageSize)
    

    并使用messageSize代替len

    建议对其他两个函数使用相同的策略,以及目前可能会溢出提供的缓冲区。

  2. 继上一个问题之后,可能是decode函数正在写入缓冲区的边界之外。将缓冲区的长度提供给decode并添加适当的检查以确保函数不会在给定范围之外写入将是一个好主意。