在 for 循环中间访问冲突写入位置 0x00000000

Question

我正在尝试将 mnist 数据集用于神经网络，但我收到访问冲突写入位置 0x00000000

密码是

for (int i = 0; i < length; i++) {
    innerarray = (int8_t*)malloc(width * height);
    for (int j = 0; j < width * height; j++) {
        int8_t value = 0;
        innerarray[j] = value;
    }
    temparray[i] = innerarray;
}
for (int i = 0; i < length; i++) {
    for (int j = 0; j < width * height; j++) {
        int8_t grayscale;
        rf.read((char*)&grayscale, 1);
        temparray[i][j] = grayscale; //error happens here
    }
}

变量值：

int length = 10000;
int width = 28;
int height = 28;

奇怪的是它只发生在 i >= 2512 时。将 grayscale 替换为 0 也不起作用。我可以在最后一个嵌套 for 循环之前将 temparray[2512][0] 设置为 0。像这样：

for (int i = 0; i < length; i++) {
        innerarray = (int8_t*)malloc(width * height);
        for (int j = 0; j < width * height; j++) {
            int8_t value = 0;
            innerarray[j] = value;
        }
        temparray[i] = innerarray;
    }
    temparray[2512][0] = 0; //works 
    for (int i = 0; i < length; i++) {
        for (int j = 0; j < width * height; j++) {
            int8_t grayscale;
            rf.read((char*)&grayscale, 1);
            temparray[i][j] = 0; //error still happens here
        }
    }

完整代码为：

#include<iostream>
#include<fstream>
#include<cstdint>
#include<cstdlib>
#include<array>

using namespace std;

struct images {
    int32_t height = 0;
    int32_t width = 0;
    int32_t magicnumber = 0;
    int32_t numberofimages = 0;
    int8_t** images[];
    void setimages(int8_t** newimages) {
        delete[] this->images;
        int8_t** images = (int8_t**)malloc(numberofimages);
        int8_t* innerarray;
        for (int i = 0; i < numberofimages; i++) {
            innerarray = (int8_t*)malloc(width * height);
            images[i] = innerarray;
        }
        for (int i = 0; i < numberofimages; i++) {
            for (int j = 0; j < width * height; j++) {
                images[i][j] = newimages[i][j];
            }
        }
    };
};

struct labels {
    int32_t magicnumber = 0;
    int32_t numberoflabels = 0;
    int8_t labels[];
};

int32_t litleendiantobig(int32_t litle) {//reverse works as well
    int32_t big = ((4278190080 & litle) >> 24) + ((255 & litle) << 24) + ((16711680 & litle) >> 8) + ((65280 & litle) << 8);
        return big;
}

images loadimages(string filename, int32_t magicalnumber) {
    ifstream rf(filename, ios::out | ios::binary);
    if (!rf) {
        cout << "Cannot open file! " << filename << endl;
        exit(1);
    }
    int32_t magicnumberoffile;
    rf.read((char*)&magicnumberoffile, 4);
    magicnumberoffile = litleendiantobig(magicnumberoffile);
    if (magicalnumber != magicnumberoffile) {
        cout << "Wrong magic number!" << endl;
        cout << "expected:" << magicalnumber << endl;
        cout << "got:" <<  magicnumberoffile << endl;
        exit(1);
    }
    images img;
    int32_t length;
    rf.read((char*)&length, 4);
    length = litleendiantobig(length);
    img.numberofimages = length;
    int32_t width;
    rf.read((char*)&width, 4);
    width = litleendiantobig(width);
    img.width = width;
    int32_t height;
    rf.read((char*)&height, 4);
    height = litleendiantobig(height);
    img.height = height;
    int8_t** temparray = (int8_t**)malloc(length);
    int8_t* innerarray;

    for (int i = 0; i < length; i++) {
        innerarray = (int8_t*)malloc(width * height);
        for (int j = 0; j < width * height; j++) {
            int8_t value = 0;
            innerarray[j] = value;
        }
        temparray[i] = innerarray;
    }
    for (int i = 0; i < length; i++) {
        for (int j = 0; j < width * height; j++) {
            int8_t grayscale;
            rf.read((char*)&grayscale, 1);
            temparray[i][j] = grayscale; //error happens here
        }
    }
    img.setimages(temparray);
    rf.close();
    return img;
}

int main() {
    images testimages;
    loadimages("t10k-images.bin", 2051);
    cout << testimages.images;
    return 0;
}

我现在不知道如何解决这个问题，也找不到其他地方。谢谢你帮我。

Answer 1

您使用 malloc 成功了。

 int* array = (int*)malloc(width* height);   // allocate width * height bytes.

 array[i] = x;  // Sets the [i] _integer_ of array to x.
                // but you allocated space for BYTE size elemennts.

使用 malloc 分配整数的正确方法：

 int* array = (int*)malloc(width* height * sizeof(int));   // allocate width * height ints

要么是那个，要么是您的初衷是分配 8 位像素。在这种情况下，您的指针应声明为 unsigned char*.

无论哪种情况，在使用 C++ 编码时，类型都很重要，使用 operator new 来分配数组会让你免于这些麻烦。

在 for 循环中间访问冲突写入位置 0x00000000

Access violation writing location 0x00000000 in middle of a for loop

access-violation

dynamic-arrays

visual-studio

c++17