Правильно ли очищается память?

@korozyan · Регистрация: 28.11.2017

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

имеется структура

C++
1
2
3
typedef struct {
    LPCWSTR letter;
} structura;

объявляю из функции саму структуру, задаю letter и вывожу сообщение

C++
1
2
3
4
structura* struc = (structura*)HeapAlloc(GetProcessHeap(), NULL, sizeof(structura));
    struc->letter = L"asdsadasd";
 
    MessageBoxW(NULL, struc->path, NULL, SW_SHOW);

после закрытия сообщения, вот такая маза:

@oleg-m1973 · 20.02.2020, 15:09

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

правильно ли очищается память?

А где ты здесь очищаешь память? Покажи нормальный код

@korozyan · 20.02.2020, 15:11 **[ТС]**

Сообщение от oleg-m1973

А где ты здесь очищаешь память? Покажи нормальный код

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

@oleg-m1973 · 20.02.2020, 15:13

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

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

Тем не менее - что такое struc->path?

@korozyan · 20.02.2020, 15:50 **[ТС]**

Сообщение от oleg-m1973

Тем не менее - что такое struc->path?

забыл изменить на letter...

@oleg-m1973 · 20.02.2020, 15:57

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

забыл изменить на letter...

Похоже, что-то ты ещё забыл. Ошибка где-то в другом месте.

@korozyan · 20.02.2020, 18:29 **[ТС]**

Сообщение от oleg-m1973

Похоже, что-то ты ещё забыл. Ошибка где-то в другом месте.

после мессадж бокса только

C++
1
ExitProcecess(0);

@oleg-m1973 · 20.02.2020, 20:00

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

после мессадж бокса только
C++Выделить код
1
ExitProcecess(0);

Ну, во-первых - я знаю случаев, когда бы мог понадобиться ExitProcecess(0) (вернее, ExitProcess - ты даже здесь умудрился неправильно написать)
Во-вторых, покажи весь код. Просто скопируй через буфер обмена, не надо своими словами пересказывать.

@korozyan · 20.02.2020, 21:20 **[ТС]**

Сообщение от oleg-m1973

Ну, во-первых - я знаю случаев, когда бы мог понадобиться ExitProcecess(0) (вернее, ExitProcess - ты даже здесь умудрился неправильно написать)
Во-вторых, покажи весь код. Просто скопируй через буфер обмена, не надо своими словами пересказывать.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
#include <windows.h>
 
typedef struct {
    LPCWSTR path;
} structura;
 
void main() {
    structura* struc = (structura*)HeapAlloc(GetProcessHeap(), NULL, sizeof(structura));
    struc->path = L"asdsadasd";
 
    MessageBoxW(NULL, struc->path, NULL, SW_SHOW);
    ExitProcess(0);
}

Добавлено через 4 минуты
если на Win 8.1 запускать, то никакой ошибки нет. Если на XP, то появляется ошибка...

@liv · 20.02.2020, 21:24

korozyan, может, все же так записать?

C++
7
int main() {

@korozyan · 20.02.2020, 21:26 **[ТС]**

странно, нор это помогло. спасибо!

@oleg-m1973 · 20.02.2020, 21:27

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

если на Win 8.1 запускать, то никакой ошибки нет. Если на XP, то появляется ошибка...

Убери ExitProcess(0); (и никогда им не пользуйся).

C++
1
2
3
4
5
6
7
8
void main() {
    structura* struc = (structura*)HeapAlloc(GetProcessHeap(), NULL, sizeof(structura));
    struc->path = L"asdsadasd";
 
    MessageBoxW(NULL, struc->path, NULL, SW_SHOW);
 
    HeapFree(GetProcessHeap(), 0, struc);
}

@korozyan · 20.02.2020, 22:35 **[ТС]**

Сообщение от oleg-m1973

Убери ExitProcess(0); (и никогда им не пользуйся).

еще вопрос, приложение крашится, когда я передаю структуру при создании потока:

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
typedef struct {
    LPCWSTR path;
} structura;
 
DWORD WINAPI ListAllDrives(LPVOID param)
{
    structura temp = *((structura*)param);
    
    MessageBoxW(NULL, temp.path, NULL, SW_SHOW);
    return 0;
}
 
void AllocateThreads() 
{
    DWORD cchBuffer;
    WCHAR* driveStrings;
    UINT driveType;
    DWORD thrid = 0;
 
    cchBuffer = GetLogicalDriveStringsW(0, NULL);
    driveStrings = (WCHAR*)HeapAlloc(GetProcessHeap(), NULL, (cchBuffer + 1) * sizeof(TCHAR));
    GetLogicalDriveStringsW(cchBuffer, driveStrings);
 
    HANDLE thr[26];
    structura* struc = (structura*)HeapAlloc(GetProcessHeap(), NULL, sizeof(structura));
    
    while (*driveStrings)
    {
        driveType = GetDriveTypeW(driveStrings);
        if (driveType == DRIVE_FIXED) {
            struc->path = driveStrings;
            thr[thrid] = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)ListAllDrives, (LPVOID)struc, 0, NULL);
            Sleep(500);
        }
        else if (driveType == DRIVE_REMOVABLE) {
            struc->path = driveStrings;
            thr[thrid] = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)ListAllDrives, (LPVOID)struc, 0, NULL);
            Sleep(500);
        }
        else if (driveType == DRIVE_REMOTE) {
            struc->path = driveStrings;
            thr[thrid] = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)ListAllDrives, (LPVOID)struc, 0, NULL);
            Sleep(500);
        }
 
        thrid += 1;
        driveStrings += lstrlenW(driveStrings) + 1;
    }
 
    for (int i = 0; i < thrid; i++) {
        WaitForSingleObject(thr[i], INFINITE);
    }
}

в чем может быть проблема?

@oleg-m1973 · 21.02.2020, 09:25

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

в чем может быть проблема?

В том, что structura* struc = (structura*)HeapAlloc надо делать перед каждым вызовом CreateThread, а в функции потока делать HeapFree

C++
1
2
3
4
5
6
7
8
9
10
11
12
      if (driveType == DRIVE_FIXED) {
 structura* struc = (structura*)HeapAlloc(GetProcessHeap(), NULL, sizeof(structura));
 
            struc->path = driveStrings;
            thr[thrid] = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)ListAllDrives, (LPVOID)struc, 0, NULL);
            Sleep(500);
        }
        else if (driveType == DRIVE_REMOVABLE) {
 structura* struc = (structura*)HeapAlloc(GetProcessHeap(), NULL, sizeof(structura));
            struc->path = driveStrings;
            thr[thrid] = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)ListAllDrives, (LPVOID)struc, 0, NULL);
            Sleep(500);

C++
1
2
3
4
5
6
7
8
DWORD WINAPI ListAllDrives(LPVOID param)
{
    structura temp = (structura*)param;
    
    MessageBoxW(NULL, temp.path, NULL, SW_SHOW);
HeapFree(.... temp);
    return 0;
}

@korozyan · 21.02.2020, 13:55 **[ТС]**

Сообщение от oleg-m1973

В том, что structura* struc = (structura*)HeapAlloc надо делать перед каждым вызовом CreateThread, а в функции потока делать HeapFree

это все хорошо, но также проблема была в функции GetLogicalDriveStringsW(); из-за нее везде нестабильно работало... переписал под GetLogicalDrives(); и все без проблем!

@Pilarentes · 21.02.2020, 14:10

Сообщение от oleg-m1973

и никогда им не пользуйся

Можно узнать, чем он так плох?

@oleg-m1973 · 21.02.2020, 14:14

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

Можно узнать, чем он так плох?

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

@Дмитрий_1982 · 21.02.2020, 14:29

Сообщение от oleg-m1973

В том, что structura* struc = (structura*)HeapAlloc надо делать перед каждым вызовом CreateThread, а в функции потока делать HeapFree

Здесь я немного не понял. А почему нужно делать так?

@oleg-m1973 · 21.02.2020, 14:33

Сообщение от Дмитрий_1982

Здесь я немного не понял. А почему нужно делать так?

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

Новые блоги и статьи Все статьи Все блоги /
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .	Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .

@korozyan 8 / 2 / 0 Регистрация: 28.11.2017 Сообщений: 157
	20.02.2020, 21:26 [ТС]
	странно, нор это помогло. спасибо! 0