Блог

Проблема решена, спасибо всем за помощь

У меня здесь небольшая проблема, это не что-то, что разрушает мою программу, но меня просто беспокоит, что я не могу это исправить. У меня есть функция, считывающая некоторые данные из файла, в конце выполнения поврежден стек вокруг переменной longGarbage. Я немного осмотрелся и обнаружил, что возможной причиной является запись в недопустимую память. Я устранил некоторые утечки памяти, которые у меня были, и проблема все еще сохраняется. Что меня смущает, так это то, что это происходит, когда функция завершает выполнение, поэтому кажется, что это происходит, когда переменная выходит за пределы области видимости. Вот код…

 CHCF::CHCF(std::string fileName)
: PAKID("HVST84838672")
{
FILE * archive = fopen(fileName.c_str(), "rb");
std::string strGarbage = "";
unsigned int intGarbage = 0;
unsigned long longGarbage = 0;
unsigned char * data = 0;
char charGarbage = '0';

if (!archive)
{
    fclose (archive);
    return;
}

for (int i = 0; i < 12; i  )
{
    fread(amp;charGarbage, 1, 1, archive);
    strGarbage  = charGarbage;
}


if (strGarbage != PAKID)
{
    fclose(archive);
    throw "Incorrect archive format";
}
strGarbage = "";

fread(amp;_gameID, sizeof(_gameID),1,archive);
fread(amp;_fileCount, sizeof(_fileCount),1,archive);

for (int i = 0; i < _fileCount; i  )
{
    fread(amp;longGarbage, 8,1,archive); //file offset

    fread(amp;intGarbage, 4, 1, archive);//fileName

    for (int i = 0; i < intGarbage; i  )
    {
        fread(amp;charGarbage, 1, 1, archive);
        strGarbage  = charGarbage;
    }

    fread(amp;longGarbage, 8, 1, archive); //fileSize

    fread(amp;intGarbage, 4, 1, archive); //fileType

    data = new unsigned char[longGarbage];

    for (long i = 0; i < longGarbage; i  )
    {
        fread(amp;charGarbage, 1, 1, archive);
        data[i] = charGarbage;
    }

    switch ((FILETYPES)intGarbage)
    {
    case MAP:
        _maps.append(strGarbage, new CFileData(strGarbage, FILETYPES::MAP, data, longGarbage));
        break;

    default:
        break;
    }

    delete [] data;
    data = 0;
    strGarbage.clear();
    longGarbage = 0;

}
fclose(archive);
} //error happens here
  

Вот конструктор CFileData:

 CFileData::CFileData(std::string fileName, FILETYPES type, unsigned char *data, long fileSize)
{
_fileName = fileName;
_type = type;
_data = new unsigned char[fileSize];

for (int i = 0; i < fileSize; i  )
    _data[i] = data[i];
}
  

1. Могу ли я предложить std::vector вместо вызова new и delete вручную? Ваш код небезопасен для исключений — происходит утечка, если генерируется исключение.

2. fread(amp;longGarbage, 8, 1, archive); //fileSize Вы уверены, что sizeof(long) равно 8? Я подозреваю, что это 4. Я полагаю, что в блоках Linux иногда это 8, но большинство других sizeof(long) равно 4, а sizeof(long long) равно 8.

3. Как насчет каких-либо конструкторов для членов этого класса? Они также могут повредить стек.

Происходит то, что что-то записывается в память вокруг или поверх местоположения longGarbage, что вызывает повреждение.

Вы не говорите, какую среду разработки вы используете. Одним из способов диагностики этого было бы установить точку останова, которая срабатывает при изменении определенного местоположения в памяти. Выберите ячейку памяти, которая перекрывает область повреждения, и дождитесь ее неожиданного срабатывания.

Другим способом диагностики этого было бы изучить код, который изменяет память вокруг или поверх longGarbage. Конечно, это может быть почти что угодно, но вероятными кандидатами являются модификации ‘data’, модификации ‘intGarbage’ и модификации самого ‘longGarbage’.

Мы можем сузить область поиска еще больше, потому что мы можем (обычно) быть достаточно уверены, что сам оператор присваивания безопасен. Подобный код data = new... вряд ли может быть виновником, поэтому на самом деле нам нужно сосредоточиться на изменениях памяти, которые включают в себя использование адреса ‘data’, ‘intGarbage’ или ‘longGarbage’. В частности, изменения в памяти, которые изменяют больше байтов, чем должны.

Несколько других уже указали, что длина long, вероятно, не равна восьми байтам. Если вы передаете fread неправильную длину, полученные дополнительные байты должны куда-то деваться.

Вы используете много магических чисел для размеров данных, поэтому я бы сначала проверил это. В частности, я сомневаюсь в этом sizeof(unsigned long)==8 и sizeof(unsigned in)==4 при всех возможных обстоятельствах. Обратитесь к документации вашего компилятора, но вам все равно следует быть осторожным, поскольку это, скорее всего, изменится с одного компилятора / платформы на другой.

Проверьте наличие этих битов:

 fread(amp;longGarbage, 8,1,archive); //file offset
  

Вы также можете захотеть использовать <iostream> библиотеку C вместо материалов C FILE* для чтения. Это позволило бы создать гораздо более короткую версию, потому что вам не нужно было бы закрывать файл 3 раза.

Из других комментариев и предоставленной информации кажется, что проблема на стороне C , вам следует использовать либо __int64 для среды Windows, либо int64_t для кроссплатформенной.