Question

只需编写一个简单的C程序来读取长二进制文件，将其分成12位块，并附加4位CRC，然后输出到另一个文件。当我在调试模式（Visual Studio 2012）中运行代码时，它返回垃圾，但是一旦我切换到发布模式，它就可以正常工作。

切换回调试模式似乎也没有带来问题......只是想了解这里发生了什么。无论VS使用“发布模式”做什么都会使代码在那里工作，但是当我使用Dev-C ++进行编译时，我总是得到与调试模式相同的垃圾。

编辑：我已经在calc函数中使用了malloc的建议更改，并在我的main中自由，但我仍然得到垃圾输出。 VS现在在跑步时崩溃了。

char * calc(char bit12[])
{
    int r0,r1,r2,r3;
    int i,tmp;
    //char * bit16;
    char * bit16 = malloc(sizeof(char)*(17));
    char bits[16];
    bit16 = &bits[0];
    /* Set registers to zero */
    r0 = r1 = r2 = r3 = 0;
    /* Insert info bits, most significative first */
    for(i=0; i<12; i++) {
        /* shift registers and make mod2 (^) sums */
        tmp = r3;
        r3 = r2;
        r2 = r1 ^ tmp;
        r1 = r0 ^ tmp;
        r0 = (bit12[i] == '0') ? 0 ^ tmp : 1 ^ tmp;
    }
    /* Insert 4 zeros to finish CRC calculation */
    for(i=0; i<4; i++) {
        /* shift registers and make mod2 (^) sums */
        tmp = r3;
        r3 = r2;
        r2 = r1 ^ tmp;
        r1 = r0 ^ tmp;
        r0 = tmp;
    }
    for (i=0;i<12;i++) 
        bit16[i]=bit12[i];
    if (r3 == 1)
        bit16[12]='1';
    else
        bit16[12]='0';
    if (r2 == 1)
        bit16[13]='1';
    else
        bit16[13]='0';
    if (r1 == 1)
        bit16[14]='1';
    else
        bit16[14]='0';
    if (r0 == 1)
        bit16[15]='1';
    else
        bit16[15]='0';

    printf("Internal Function input: %s\nInternal Function output: %s\n", bit12, bit16);

    return bit16;

}


int main(){
    char str[999];
    char *os,outstr[16];
    FILE *dataIn, *dataOut;
    os = &outstr[0];

    dataIn = fopen("data.txt", "r");
    dataOut = fopen("out.txt", "w");

    if(dataIn){
        //printf("test point 1.\n");
        while(fscanf(dataIn, "%s", str)!= EOF){
            os = calc(str);
            fprintf(dataOut,"%0.16s\n", os);
            free(os);
            //printf("input: %s\n", str);
            //printf("output1: %s\n", outstr);
            //printf("output2: %s\n", os);
            system("PAUSE");
        }
        fclose(dataIn);
        fclose(dataOut);
    }else{
        printf("Error opening data!\n");
    }

    printf("end of prog.\n");
    return 0;
}

Answer 1

garbage in “Debug” mode but not in “Release” mode

相信我，这与此毫无关系。

char bits[16];是一个具有自动存储持续时间的Local（数组）变量，一旦它离开其声明范围，即当calc函数返回时，对它的引用将变为无效。当函数结束时，数组不再存在，当您稍后访问该指针/地址时Undefined behavior。

而不是将本地数组放在堆栈上，malloc它：

char * bit16 = malloc(sizeof(char)*(17)); //+1 for NUL char 
//do something
return bit16;

当您不需要时，使用free清除内存。

os = calc(str);
fprintf(dataOut,"%0.16s\n", os);
free(os);

Answer 2

您正在返回指向局部变量的指针。虽然这可能会欺骗编译器

char * bit16;
char bits[16];

返回bit16时无效。

因为数据存储在calc()函数的堆栈帧中，因此当函数返回时它将被销毁。

您需要使用malloc()和memcpy()这样的

复制值

char *bit16 = malloc(16);
if (bit16 == NULL)
    return NULL;
memcpy(bit16, bits, 16);

你应该小心，因为bit16数组不能用作字符串，因为它没有'\0'终结符。

然后，您需要在free()

的返回值上调用calc()

为了避免所有这些，你可以将数组作为参数传递给函数，就像这个

char * calc(char bit12[], char bit16[])

然后在调用函数

中

char * calc(char bit12[], char bits[])

char bit16[16];
calc(str, bit16);

没有返回值，您当然应该删除bits函数中bit16和calc()的声明。

Answer 3

您的bits[]数组是函数的本地数组，即在返回函数后释放它。因此返回指针中的值可以被程序的其他部分覆盖。所以很可能你会得到垃圾。有时候（比如在你的发行版中）你可以从中获得幸运并获得有用的东西。但这完全不可靠。

Answer 4

我做C编程已经很长时间了，但对我来说，似乎你从calc（）返回一个错误的指针。 bit16指向bits16 [0]，但在函数结束时这将变为无效。

所以你在两次调用之间的短暂时间内有一个无效指针：

 os = calc(str);
 /* here, os points to freed memory */
 fprintf(dataOut,"%0.16s\n", os);

我相信，如果您只是在这两行之间添加内容，消耗并释放内存，您将能够使您的错误更具可重复性。

bits16 [16]应该是calc之外的静态char数组。

只是一枪。

C函数返回char指针在＆＃34; Debug＆＃34;模式但不在＆＃34;发布＆＃34;模式

4 个答案: