c - 在 C 中读取二进制文件期间检测到堆栈粉碎

Question

底部更新====

不久前，我创建了以下函数，我成功地使用它从转换为 .bin 文件的图像(w x h 尺寸)中获取灰度值。它只是给出所有像素值的数组。但是，它并不是像这样的函数，而是立即放入 main() 中。

// read the BIN-file as grayscale image
void decodeBIN(const char* filename, short image[], int w, int h){
    int i = 0;
    unsigned char buffer[16]; // no specific size attributed
    FILE *ptr;

    ptr = fopen(filename, "rb");
    if (!ptr){
        printf("\nUnable to open file!\n"); // error
    }

    while (!feof(ptr)){
        fread(buffer,2,1,ptr); // read w bytes to buffer
        image[i] = buffer[1];
        //printf("%u ", image[i]); // DEBUG
        i++;
    }
    fclose(ptr);
    printf("\nBinary image read (npixels: %i).\n", i-1); // DEBUG
}

我决定扩展代码，所以我将其重写为之前的函数，并将其放在单独的函数文件中，并且还制作了头文件。函数的额外文件和头文件可以 100% 工作，所以这不是问题。现在，这段代码不再起作用，并且我收到堆栈粉碎错误。在这个函数之后调用的一些变量也跳转到另一个值，所以我认为问题出在缓冲区(我不知道缓冲区的正确大小，但它有效......)。经过一些实验和测试，我想出了以下功能。我用名为 image2 的字符数组替换了缓冲区，以便简单地尝试测试它:

void decodeBIN(const char* filename, short image[], int w, int h){
    int i = 0, res;
    char image2[];
    FILE *ptr;

    ptr = fopen(filename, "rb"); //"MySnap_20180327-2239-010.bin"
    if (!ptr){
        printf("\nUnable to open file!\n"); // error
    }

    res = fread(image2,1,w*h,ptr) // need to read w*h pixels

    while (i < w*h){ // DEBUG
        printf("%i ", (int)image2[i]); // DEBUG
        i++;
    }
    printf("\nRead %u bytes\n", res); // DEBUG
    fclose(ptr);
    printf("Binary image read (npixels: %i).\n", i); // DEBUG
}

我对它以前的工作方式有点迷失，突然当我将代码从 main() 移动到一个函数时它停止工作，所以欢迎任何帮助!

提前致谢。

免责声明:我的目标是在尽可能少的库的帮助下编写此内容

=====更新:

在 @alainmerigot 回答之后，我得到了这段代码，这有助于获取正确的值:

void decodeBIN(const char* filename, unsigned char image[], int w, int h){
    int i = 0, res;
    FILE *ptr;

    res = fread(image,sizeof(char),w*h,ptr) // need to read w*h pixels
    fclose(ptr);
}

尽管如此，段错误和跳转变量仍然存在，因此这里对我正在做的事情进行了更高层的监督:

char filenamePathed["[path of file]/file.bin"];
short img1[npixels]; // npixels = w*h
printf("i_file: %i\n", i_file); // correct value
decodeBIN(filenamePathed, img_curr, w, h); // decode bin
printf("i_file: %i\n", i_file); // value jumped
while (i < npixels){
    img1[i] = (short)img_curr[i];
    i++;
}

也许很高兴知道我正在对多个文件(时间序列)迭代执行此操作？我还需要它以(短)格式(或整数，但最终需要内存效率高，像素的范围为 0-255，所以 int 有点丰富)结束。

Answer 1

第二个函数的问题是您在数组 image2 中写入，而没有为其保留空间。声明 char image2[]; 仅表示存在一个数组，并且可以在 var image2 中找到该数组的地址，但没有空间与之关联，因此问题。

您可以通过多种方式将空间与该数组关联起来。

使用堆中的永久存储

image2 = malloc(x*y);//但不要忘记在函数末尾释放(image2)

使用堆栈中的临时存储(离开函数时会自动释放空间)。

image2 = alloca(x*y);//比 malloc 稍快，并且不需要 free() 图像

但最好是使用具有参数化大小的数组(自 C99 起)。您的数组应声明为

char image2[w*h];//将使用 w 和 h 的值来定义数组大小

如果您想要在函数中打印图像值之外的其他操作，则应该将图像存储在永久内存中，并有办法知道程序中数组的地址。这可能就是您想要的，也是您的参数列表中包含 short image[] 的原因。

解决方案只是简单地在 fread() 中使用 image 而不是 image2。

但是，image 的声明应该是连贯的，并且 image 应该是 char 数组而不是 short。

还要注意声明。在第一个函数中，图像是一个 unsigned char 数组，在第二个函数中，图像是一个 char 数组。虽然存储大小相同并且 fread() 将存储相同的值，但它们并不等效。如果在算术上下文中使用，image[i] 将被不同地解释，并且结果可能会不同。一般来说，图像是无符号的。