0

它因调试错误而崩溃,并说变量“代码”周围的堆栈已损坏。这是我正在做的汉明代码实验室的代码。输入文件只是同一行上的一堆 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];

}
4

1 回答 1

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并添加适当的检查以确保函数不会超出给定的范围写入将是一个好主意。

于 2013-07-28T21:03:47.930 回答