HEAP CORRUPTION DETECTED

@Taba4nik · Регистрация: 04.06.2009

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

После нажатия на Выход (IDC_Exit) если до этого выводилась матрица (IDC_Form) выдает ошибку HEAP CORRUPTION DETECTED

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
#include <stdio.h> 
#include <windows.h>
#include "resource.h"
#include <direct.h>
#include <cmath>
#include <malloc.h>
#include <crtdbg.h>
 
char    file_name[MAX_PATH+1],customfilter[100];
 
OPENFILENAME of, sf;
 
int elem, cep;
int **matr; //Матрица смежности графа Кенига
char buf[10000], buff[10000];//Промежуточные буферы для храниния информации выводимой на экран
int f;
 
struct spisok //Динамический список
{
    int a,b;    
    spisok*next;            
};
spisok *first=NULL;
int vvod_fail() //Ввод из файла
{
    char initialdir[MAX_PATH+1];
    getcwd(initialdir,MAX_PATH+1);
    memset(&of,0,sizeof(of));
    of.lStructSize=sizeof(of);
    of.lpstrFilter= "Текстовые файлы (*.txt)\0*.txt\0" 
       "Все файлы (*.*)\0*.*\0";      
    of.nFilterIndex=1;
    sprintf(customfilter,"Использованный фильтр");
    of.lpstrCustomFilter=customfilter;
    of.nMaxCustFilter=100;
    file_name[0]='\0';
    of.lpstrFile=file_name;
    of.nMaxFile=MAX_PATH+1;
    of.lpstrInitialDir=initialdir;
    of.lpstrTitle="Диалог открытия файла";
if(GetOpenFileName(&of)!=0)
    {
        int a,b;    
        FILE*f_in;
        f_in=fopen(of.lpstrFile,"r");
        if(f_in!=NULL)// Если файл существует
        {
            spisok *tu=NULL;
            spisok *pu=NULL;
            while(fscanf(f_in,"%d %d",&a,&b)==2) //Чтение информации в список из файла
            {
                tu=new spisok;
                if(first==NULL) first=tu;
                else pu->next=tu;
                pu=tu;
                tu->a=a;
                tu->b=b;
            }
            if(pu!=NULL) pu->next=NULL;
            fclose(f_in);
            return 0;
        }
        else
        return 1;
        delete of.lpstrFile;
    }
}
void form_matr() //Формирование матрицы смежности графа Кенига
{
    int n, i, cc;
    spisok *tu;
    tu=first;
    while(tu!=NULL)
    {
        n=tu->b;
        tu=tu->next;
        for (i=0;i<n;i++)
        {
            if(elem<tu->a) elem=tu->a; //Подсчет кол-ва элементов
            tu=tu->next;            
        }
        cep++; //Подсчет кол-ва цепей
    }
    matr=(int **)calloc(cep,sizeof(int *)); //Выделение памяти под матрицу смежности
    for (i=0; i<=cep; i++) matr[i]=(int *)calloc(elem,sizeof(int));
    tu=first;
    while(tu!=NULL)
    {
        n=tu->b;
        cc=tu->a-1;
        tu=tu->next;
        for (i=0;i<n;i++)
        {
            matr[cc][tu->a-1]=1;
            tu=tu->next;            
        }
    }
}
void save_fail()
{
   char initialdir[MAX_PATH+1];
   getcwd(initialdir,MAX_PATH+1);
   memset(&sf,0,sizeof(sf));
   sf.lStructSize=sizeof(sf);
   sf.lpstrFilter=   "Текстовые файлы (*.txt)\0*.txt\0"
                            "Все файлы (*.*)\0*.*\0";
   sf.nFilterIndex=1;
   sprintf(customfilter,"Использованный фильтр");
   sf.lpstrCustomFilter=customfilter;
   sf.nMaxCustFilter=100;
   file_name[0]='\0';
   sf.lpstrFile=file_name;
   sf.nMaxFile=MAX_PATH+1;
   sf.lpstrInitialDir=initialdir;
   sf.lpstrTitle="Диалог сохранения файла";
if(GetSaveFileName(&sf)!=0)
    {
        FILE*f_out;
        f_out=fopen(sf.lpstrFile,"w");
        if(f_out!=NULL)
        {
            fprintf(f_out,"Элементы матрицы\n");
            for(int i=0;i<cep;i++)
            {fprintf(f_out," \n");
             for(int j=0;j<elem;j++)
             {
                 fprintf(f_out,"%d",matr[i][j]);
                 fprintf(f_out," ");
             }
            }
            fclose(f_out);
        }
    }
}
void vivod_din_sp()
{
    char *r=buff;
    int l=0, n;
    spisok *tu;
    tu=first;
    while(tu!=NULL)
    {
        l=sprintf(r,"%s %d %s %d\r\n","Цепь  ",tu->a,"      |Кол-во элементов ",tu->b);
        r+=l;
        n=tu->b;
        tu=tu->next;
        for (int i=0;i<n;i++)
        {
            l=sprintf(r,"%s %d %s %d\r\n","Элемент  ",tu->a,"|Контакт ",tu->b);
            tu=tu->next;
            r+=l;
        }
    }
}
 
 
void vivod_matr()
{
    char *r=buf;
    int l=0;
    for(int i=0;i<cep;i++)
        {
            for(int j=0;j<elem;j++)
                {
                    l=sprintf(r,"%4d",matr[i][j]);
                    r+=l;
                }
            r[0]='\r';
            r[1]='\n';
            r+=2;
        }   
}
void del()
{
    if(first!=NULL)
    {
        for(int i=0;i<cep;i++)
        {
            free(matr[i]);
        }
        free(matr);
    }
    if(first!=NULL)
    {
        spisok *pu=NULL;
        while (first!=NULL)
        {
            pu=first->next;
            delete first;
            first=pu;
        }
    }
}
int CALLBACK MyDlgProc(HWND hwnd,UINT msg,WPARAM wp,LPARAM lp)
{
    const double pi=3.1415;
    int r=120;
    
    switch(msg)             
    {
        case WM_INITDIALOG:break;   
        case WM_COMMAND:        
            switch(LOWORD(wp))                                  
            {
                case IDC_Exit:
                        return EndDialog(hwnd,IDC_Exit);
                case IDC_Form:                          
                    {
                        if(first!=NULL)
                        {
                        form_matr();
                        vivod_matr();
                        SetWindowText(GetDlgItem(hwnd,IDC_Matr),buf);
                        }
                        else
                        MessageBox(hwnd,"Матрица не может быть сформирована(Список пуст)","Ошибка",MB_OK);
                        break;
                    }
                case IDC_Open:
                    {
                        del();
                        if(vvod_fail()==1) MessageBox(hwnd,"Фаил не существует","Ошибка",MB_OK);
                        break;
                    }
                case IDC_Graph:
                    {
                        f=1;
                        RECT rect;
                        rect.top=170;
                        rect.left=230;
                        rect.right=600;
                        rect.bottom=430;
                        InvalidateRect(hwnd,&rect,1);
                        break;
                    }
                case IDC_Out:
                    {
                        if(first!=NULL)
                        {
                        vivod_din_sp();
                        SetWindowText(GetDlgItem(hwnd,IDC_Spis),buff);
                        }
                        else MessageBox(hwnd,"Не загружены исходные данные","Ошибка",MB_OK);
                        break;
                    }
                case IDC_Save:save_fail();break;
                default:return 0;                           
            }
            break;
    case WM_PAINT:
        int *xe,*ye, *xc, *yc;
        int i;
        char str[10];
        HDC hdc;
        HPEN hpen,old_pen;
        HBRUSH hbrush,old_brush;
        PAINTSTRUCT ps;
            hdc=BeginPaint(hwnd,&ps);
            if((first!=NULL)&&(f==1))
            {
                xe=(int*)_alloca(elem*sizeof(int));
                ye=(int*)_alloca(elem*sizeof(int));
                xc=(int*)_alloca(cep*sizeof(int));
                yc=(int*)_alloca(cep*sizeof(int));
                SetBkMode(hdc,TRANSPARENT);
                hpen=CreatePen(PS_SOLID,1,RGB(0,0,255));
                hbrush=CreateSolidBrush(RGB(255,0,0));
 
                for(i=0;i<elem;i++)
                {
                    xe[i]=300+i*35;
                    ye[i]=250;
                }
                for(i=0;i<cep;i++)
                {
                    xc[i]=300+i*35;
                    yc[i]=350;
                }
                old_pen=(HPEN)SelectObject(hdc,hpen);
                old_brush=(HBRUSH)SelectObject(hdc,hbrush);
                for(int n=0;n<cep;n++)
                {
                    for(int m=0;m<elem;m++)
                    {
                        if(matr[n][m]>0)
                        {
                            MoveToEx(hdc,xc[n],yc[n],0);
                            LineTo(hdc,xe[m],ye[m]);
                        }
                    }
                }
                for(i=0;i<elem;i++)
                {
                    sprintf(str,"%d",i+1);
                    Ellipse(hdc,xe[i]-15,ye[i]-15,xe[i]+15,ye[i]+15);
                    TextOut(hdc,xe[i]-5,ye[i]-5,str,1) ;
                }
                for(i=0;i<cep;i++)
                {
                    sprintf(str,"%d",i+1);
                    Ellipse(hdc,xc[i]-15,yc[i]-15,xc[i]+15,yc[i]+15);
                    TextOut(hdc,xc[i]-5,yc[i]-5,str,1) ;
                }
                EndPaint(hwnd,&ps);
                DeleteObject(hpen);
                DeleteObject(hbrush);
            }
            break;
        default:return 0;
    }
    return 1;
}            
int WINAPI WinMain( HINSTANCE hInstance,      
                    HINSTANCE hPrevInstance,  
                    LPSTR lpCmdLine,          
                    int nCmdShow)
{
    DialogBox(0,MAKEINTRESOURCE(IDD_DIALOG1),0,MyDlgProc);
    del();
}

@XuTPbIu_MuHTAu · 19.07.2009, 16:05

извини,кода многовато,чтобы за тебя искать ошибку. Где-то ты ошибся с выделением памяти. Проверяй.

@IT_Exp · 19.07.2009, 16:12

Taba4nik,
ознакомься с содержимым заголовочного файла assert.h и/или crtdbg.h

@Patch · 19.07.2009, 17:04

Taba4nik,
del() как-то странно написан: чтобы два раза подряд проверять состояние одной и той-же переменной, не изменяемой между этими проверками? это зачем???

if(first!=NULL)

кстати, вот этот кусок точно работать не будет:

for(int i=0;i<cep;i++)
{
free(matr[i]);
}
free(matr);

потому, что matr = matr[0], а его уже удалили, при проходе цикла.

@XuTPbIu_MuHTAu · 19.07.2009, 18:01

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

потому, что matr = matr[0], а его уже удалили, при проходе цикла.

эээ???
matr - это указатель на matr[0], никак не он сам
он же не пишет free(*matr).Это действительно была бы ошибка

@Taba4nik · 19.07.2009, 19:26 **[ТС]**

спасибо, за помощь

C++
1
2
3
4
5
for(int i=0;i<cep;i++)
{
free(matr[i]);
}
free(matr);

убрал последнюю строчку и все заработало. даже неловко как-то

@XuTPbIu_MuHTAu · 19.07.2009, 19:31

Taba4nik, ошибка не в этом,хотя кажется,что ты ее исправил. На самом деле вот она ошибка

Code
1
2
matr=(int **)calloc(cep,sizeof(int *)); //Выделение памяти под матрицу смежности
 for (i=0; i<=cep; i++) matr[i]=(int *)calloc(elem,sizeof(int));

в цикле for ты выделяешь один лишний элемент, там должно быть i< cep. Ошибка исчезла когда ты убрал последнюю строку только потому,что ты не очищаешь память выделенную под массив указателей,формирующих строки матрицы. В итоге все просто : ты записывал лишний элемент за границу,чем и вызывал ошибку кучи.

@Patch · 20.07.2009, 06:45

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

matr - это указатель на matr[0], никак не он сам

согласен.
тем не менее, компилятор MSVS с такой конструкцией работает не всегда.
я, вообще-то, alloc/free не пользуюсь, но и с new/delete подобные проблемы возникают.
причем только с массивами указателей.
в подобных случаях удалял нужный блок через явное преобразование типов:
delete (void *)xxx; (или free((void *)matr); в данном случае)

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

убрал последнюю строчку и все заработало

а вот это не есть хорошо.
хип очищать все равно надо.
если выделял память - ее нужно освободить, иначе в большом приложении будут проблемы с утечками.

и еще одно замечание:
я тут прикинул, что будет, если я решу последовательно два файла загрузить и посмотреть на картинке.
не выходя из диалога.
и получилась фигня, потому, что переменные cep и elem при открытии файла или при формировании матрицы не обнуляются.
весьма частая ошибка.

собственно, я вообще инициализации их не нашел. может и смотрел плохо.

@Taba4nik · 23.07.2009, 21:17 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@XuTPbIu_MuHTAu 2256 / 771 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	19.07.2009, 16:05
	извини,кода многовато,чтобы за тебя искать ошибку. Где-то ты ошибся с выделением памяти. Проверяй. 1

@IT_Exp MCSD: APP BUILDER 8795 / 1074 / 104 Регистрация: 17.06.2006 Сообщений: 32,602
	19.07.2009, 16:12
	Taba4nik, ознакомься с содержимым заголовочного файла assert.h и/или crtdbg.h 1

@Taba4nik 1 / 1 / 0 Регистрация: 04.06.2009 Сообщений: 6
	23.07.2009, 21:17 [ТС]
	спасибо, за помощь. довел программу до ума. осталась маленькая помарочка. при нажатии на крестик наверху окна диалог не зыкрывается. подскажите, пожалуйста, как исправить 0