从X长度的文件中读取二进制数据时,C ++程序崩溃

时间:2013-07-03 20:46:37

标签: c++ file binary fstream

我正在尝试从文件中读取二进制数据,这是我的文件结构:

#define SIGNATURE_LENGTH 3
#define VERSION_LENGTH 2
#define DATACOUNT_LENGTH 4
#define COMPRESS_LENGTH 1
#define FORMAT_LENGTH 2
#define DATALENGTH_LENGTH 4

const unsigned char resSignature[SIGNATURE_LENGTH] = { 0x52, 0x45, 0x53 };
const unsigned char resVersion[VERSION_LENGTH] = { 0x01, 0x00 };

const unsigned char isCompressed[COMPRESS_LENGTH] = { 0x01 };
const unsigned char notCompressed[COMPRESS_LENGTH] = { 0x00 };

// Data Formats:
const unsigned char dataUnknown[FORMAT_LENGTH] = { 0x00, 0x00 };
const unsigned char dataXML[FORMAT_LENGTH] = { 0x01, 0x00 };


// Define header structure for resource file
struct ResHeader
{
    unsigned char signature[SIGNATURE_LENGTH];
    unsigned char version[VERSION_LENGTH];
};

// Define data structure for resource file
struct ResData
{
    unsigned char compressed[COMPRESS_LENGTH];
    unsigned char dataFormat[FORMAT_LENGTH];
    unsigned char dataLength[DATALENGTH_LENGTH];
    unsigned char *data;
};

我的班级使用:

std::fstream File;

// Resource file makeup
ResHeader  header;
unsigned char dataCount[DATACOUNT_LENGTH];

// Vector to contain resource file data
std::vector<ResData> ResourceData;

当我尝试从文件中读取时程序崩溃:

int ResourceFile::LoadFile()
{
    File.open("blah.dat", std::ios::in | std::ios::binary);

    // Read header
    File.read((char*) header.signature, SIGNATURE_LENGTH);
    File.read((char*) header.version, VERSION_LENGTH);
    if(!VerifyHeader())
    {
        File.close();
        return HEADER_INCORRECT;
    }
    File.read((char*) dataCount, DATACOUNT_LENGTH);
    long fileCount = unsignedCharArrayToLong(dataCount);
    for(long i = 0; i < fileCount; ++i)
    {
        ResData tmp;
        File.read((char*) tmp.compressed, COMPRESS_LENGTH);
        File.read((char*) tmp.dataFormat, FORMAT_LENGTH);
        File.read((char*) tmp.dataLength, DATALENGTH_LENGTH);
        File.read((char*) tmp.data, unsignedCharArrayToLong(tmp.dataLength));
        ResourceData.push_back(tmp);
    }
    File.close();
    return SUCCESS;
}

程序崩溃了:

File.read((char*) tmp.data, unsignedCharArrayToLong(tmp.dataLength));

文件中数据的长度是282,这是读入tmp.dataLength的内容;所以数字是准确的。还使用easy zlib压缩数据:http://www.firstobject.com/easy-zlib-c++-xml-compression.htm

对于我做错了什么或者我能做得更好的任何建议/帮助将不胜感激。 谢谢。

1 个答案:

答案 0 :(得分:2)

这个局部变量:

ResData tmp;

包含

unsigned char *data;

鉴于没有任何代码实际上为data分配任何内容,它将指向某些&#34;随机&#34;放在记忆中。这意味着&#34;未定义的行为&#34;,并给出平均律和您的结果,在这种情况下&#34;未定义的行为&#34;意味着你的程序崩溃,这可能比替代方案更好,或者当它在其他地方出错时你会更多地挠头。

你可能想要这样的东西(在阅读dataLength之后):

size_t len = unsignedCharArrayToLong(tmp.dataLength); 

tmp.data = new unsigned char[len];
File.read((char*) tmp.data, len);

稍后,不要忘记取消分配数据。或者更好的是,使用std::vector,而不是调用new,执行data.resize(len);并使用tmp.data.data()获取File.read(...)中缓冲区的地址。这样,您就不需要记住取消分配任何内容,因为std::vector会为您执行此操作。