Heap corruption при вызове delete[]

@NkL · Регистрация: 11.10.2020

Студворк — интернет-сервис помощи студентам

Имеется двумерный список, каждый элемент содержит несколько полей (string, int и простой класс). Требуется реализовать считывание и запись в файл всей структуры. Для записи string в файл я записываю сначала размер строки, потом указатель на сам массив. При считывании выделяю буфер, записываю в него данные, копирую в обёртку string данные для дальнейшей работы, а потом удаляю буфер. Почему при удалении буфера вылетает ошибка heap corruption? Пробовал выяснить, в чём причина. При увеличении размера буфера ошибка пропадает. С чем это связано?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
ostream& Warehouse::writeToBin(ostream& stream) {
    Element* current = head;
    while (current != nullptr) {
        int nameLen = current->data.name.length();
        stream.write((char*)&nameLen, sizeof(int));
        stream.write((char*)current->data.name.data(), sizeof(char*));
        current = current->next;
    }
    return stream;
}
 
istream& Warehouse::readFromBin(istream& stream) {
    Element* current = head;
    while (head != nullptr) {
        current = current->next;
        delete head;
        head = current;
    }
    Product product;
    while (!stream.eof()) {
        string str;
        int nameLen;
        stream.read((char*)&nameLen, sizeof(int));
        char* nameBuf = new char[nameLen + 1];
        stream.read(nameBuf, sizeof(char*));
        str = nameBuf;
        cout << str;
        delete[] nameBuf;
        //Add(product);
    }
    return stream;
}

Алексей1153 · 12.08.2021, 12:46

попробуй

Сообщение от NkL

stream.write((char*)current->data.name.data(), sizeof(char*));

->

C++
1
stream.write(current->data.name.c_str(), current->data.name.size());

Сообщение от NkL

char* nameBuf = new char[nameLen + 1];
stream.read(nameBuf, sizeof(char*));
str = nameBuf;
cout << str;
delete[] nameBuf;

->

C++
1
2
current->data.name.resize(nameLen,'\0');
stream.read(current->data.name.data(), current->data.name.size());

Добавлено через 1 минуту

Сообщение от NkL

int nameLen

int нужно менять на size_t

Сообщение от NkL

while (!stream.eof()) {

проверка !stream производится после чтения, а не до. И eof не нужен

@DrOffset · 12.08.2021, 13:02

Не по теме:

Сообщение от NkL

read(nameBuf, sizeof(char*));

Всегда удивляло настойчивое желание считать указатель чем-то большим, чем он есть.
Поделитесь, откуда вы это берете, чтобы я людям не советовал туда больше ходить :)

@NkL · 12.08.2021, 18:10 **[ТС]**

Сообщение от Алексей1153

попробуй

Такой вариант вроде без ошибок работает. Для теста сделал string переменную.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
ostream& Warehouse::writeToBin(ostream& stream) {
    Element* current = head;
    while (current != nullptr) {
        int nameLen = current->data.name.length();
        stream.write((char*)&nameLen, sizeof(int));
        stream.write((char*)current->data.name.c_str(), current->data.name.size());
        current = current->next;
    }
    return stream;
}
 
istream& Warehouse::readFromBin(istream& stream) {
    Element* current = head;
    while (head != nullptr) {
        current = current->next;
        delete head;
        head = current;
    }
    Product product;
    while (stream) {
        string str;
        int nameLen;
        stream.read((char*)&nameLen, sizeof(int));
        str.resize(nameLen, '\0');
        stream.read((char*)str.data(), str.size());
        cout << str;
        //Add(product);
    }
    return stream;
}

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

Я же правильно понял, что такая логика будет работать до тех пор, пока программа не считает весь файл?

C++
1
2
3
    while (stream) {
        ...
    }

Алексей1153 · 12.08.2021, 19:52

NkL, подправил в двух местах между комментариями "начало исправлений" и "конец исправлений"

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
ostream& Warehouse::writeToBin(ostream& stream) {
    Element* current = head;
    while (current)
    {
        //начало исправлений
        //
        const auto& name=current->data.name;
        const size_t Len = name.size();
        stream.write((const char*)&Len, sizeof(Len));
        stream.write(name.c_str(), Len);
        current = current->next;
        //
        //конец исправлений
    }
    return stream;
}
 
istream& Warehouse::readFromBin(istream& stream) {
    Element* current = head;
    while (head)
    {
        current = current->next;
        delete head;
        head = current;
    }
    
    Product product;
    //начало исправлений
    //
    do
    {
        string str;
 
        size_t Len{};
        stream.read((char*)&Len, sizeof(Len));
        if(!stream)break;
 
        try{str.resize(Len, '\0');}catch(...){break;}
        stream.read(str.data(), str.size());
 
        cout << str;
        //Add(product);
    }while(stream);
    //
    //конец исправлений    
    return stream;
}

@TheCalligrapher · 12.08.2021, 20:05

Сообщение от NkL

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

Разумеется. Как же еще? Распространенные ошибки

Добавлено через 1 минуту

Сообщение от NkL

Я же правильно понял, что такая логика будет работать до тех пор, пока программа не считает весь файл?

Если быть точным, то такая логика будет работать до тех пор, пока поток не перейдет в состояние fail. Которая в том числе может быть вызвана тем, что сделана попытка чтения данных после конца файла. Но не только этим.

@NkL · 13.08.2021, 11:53 **[ТС]**

Написал такой код для своей задачи

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
ostream& Warehouse::writeToBin(ostream& stream) {
    Element* current = head;
    while (current) {
        //const auto& data = current->data;
        int nameLen = current->data.name.size(), categoryLen = current->data.category.size();
        stream.write((char*)&nameLen, sizeof(int));
        stream.write((char*)current->data.name.c_str(), nameLen);
        stream.write((char*)&categoryLen, sizeof(int));
        stream.write((char*)current->data.category.c_str(), categoryLen);
        stream.write((char*)&current->data.count, sizeof(int));
        stream.write((char*)&current->data.arrivalDate, sizeof(Product::Date));
        stream.write((char*)&current->data.cost, sizeof(int));
        stream.write((char*)&current->data.percent, sizeof(int));
        current = current->next;
    }
    return stream;
}
 
istream& Warehouse::readFromBin(istream& stream) {
    Element* current = head;
    while (head != nullptr) {
        current = current->next;
        delete head;
        head = current; 
    }
    while (stream) {
        int count, cost, percent, nameLen, categoryLen;
        Product::Date arrivalDate;
        string name, category;
        stream.read((char*)&nameLen, sizeof(int));
        name.resize(nameLen, '\0');
        stream.read((char*)name.data(), name.size());
        stream.read((char*)&categoryLen, sizeof(int));
        category.resize(categoryLen, '\0');
        stream.read((char*)category.data(), category.size());
        stream.read((char*)&count, sizeof(int));
        stream.read((char*)&arrivalDate, sizeof(Product::Date));
        stream.read((char*)&cost, sizeof(int));
        stream.read((char*)&percent, sizeof(int));
        Product product(name, category, count, arrivalDate, cost, percent);
        Add(product);
    }
    return stream;
}

Всё отлично работает, за исключением одного нюанса. После прочтения всех моих объектов, цикл срабатывает ещё один раз. В конец структуры читается ещё один элемент, идентичный прошлому. В чём может быть ошибка? В самом файле записано 3 объекта, но по итогу работы - 4 элемента.

Алексей1153 · 13.08.2021, 12:01

NkL, прочитай внимательно пост №5 и возьми оттуда все исправления

@NkL · 13.08.2021, 13:21 **[ТС]**

Алексей1153, Как ни странно, данная строчка вашего кода решила мою проблему

C++
1
if(!stream)break;

Видимо, istream принимает '\0' за данные, поэтому дополнительная проверка в цикле и требуется.

Алексей1153 · 13.08.2021, 13:53

NkL, на самом деле, особое внимание нужно было обратить на:

Сообщение от Алексей1153

//начало исправлений
//
do
{
string str;
size_t Len{};
stream.read((char*)&Len, sizeof(Len));
if(!stream)break;
try{str.resize(Len, '\0');}catch(...){break;}
stream.read(str.data(), str.size());
cout << str;
//Add(product);
}while(stream);
//
//конец исправлений

но и остальные исправления тоже. А ты упорно игнорируешь )

@NkL · 13.08.2021, 14:08 **[ТС]**

Алексей1153, Я когда отлаживал программу, менял свой цикл на do-while, ничего не поменялось. Суть этого цикла же в другом - выполнять самую первую итерацию в любом случае, даже когда условие цикла не выполняется.

Алексей1153 · 13.08.2021, 14:11

NkL, ну, ежели подсказки тебе не интересны, разбирайся сам

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

Алексей1153 фрилансер 6444 / 5639 / 1128 Регистрация: 11.10.2019 Сообщений: 15,003
	13.08.2021, 12:01
	NkL, прочитай внимательно пост №5 и возьми оттуда все исправления 0

@NkL 1 / 1 / 0 Регистрация: 11.10.2020 Сообщений: 59
	13.08.2021, 14:08 [ТС]
	Алексей1153, Я когда отлаживал программу, менял свой цикл на do-while, ничего не поменялось. Суть этого цикла же в другом - выполнять самую первую итерацию в любом случае, даже когда условие цикла не выполняется. 0

Алексей1153 фрилансер 6444 / 5639 / 1128 Регистрация: 11.10.2019 Сообщений: 15,003
	13.08.2021, 14:11
	NkL, ну, ежели подсказки тебе не интересны, разбирайся сам 0

Heap corruption при вызове delete[]

Решение