Странное поведение при передаче структуры в поток

@ids1488 · Регистрация: 07.09.2019

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

Добрый день! Долгое время пишу только на С# и Qt. Мозги совсем атрофировались от низкоуровневого кода. Но пришлось решать задачу с малыми ресурсами и максимальной скоростью. Математику задачи решил и реализовал. Но потребовалось асинхронное выполнение некоторых функций - сетевой копирование файлов, объемом > 2 Тбайт. Решил выкинуть в поток. И тут возникла проблема.

Структура для потока:

C++
1
2
3
4
5
6
7
struct ARGS
{
    TCHAR pszFileSrc[MAX_PATH];
    TCHAR pszFileDst[MAX_PATH];
    HWND m_test;
    HWND m_Main;
};

Кусок кода потока:

C++
1
2
3
4
5
6
...
ARGS *m_args = (ARGS*)p_Args;
CEnsureCloseFile hFileSrc = CreateFile(m_args->pszFileSrc, GENERIC_READ,
         FILE_SHARE_READ, NULL, OPEN_EXISTING, 
         FILE_FLAG_NO_BUFFERING | FILE_FLAG_OVERLAPPED, NULL);
...

После вызова данной функции данные структуры m_args изменяются на черте что.

Но если сделать вот так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
ARGS *m_args = (ARGS*)p_Args;
 
   TCHAR szPathname[_MAX_PATH];
   TCHAR szSavePathname[_MAX_PATH];
   TCHAR szRenamePathname[_MAX_PATH];
   HWND m_hwnd = m_args->m_test;
   HWND m_MainHwnd = m_args->m_Main;
 
   _stprintf(szPathname, _T("%s"), m_args->pszFileSrc);
   _stprintf(szSavePathname, _T("%s"), m_args->pszFileDst);
 
   CEnsureCloseFile hFileSrc = CreateFile(szPathname, GENERIC_READ,
         FILE_SHARE_READ, NULL, OPEN_EXISTING, 
         FILE_FLAG_NO_BUFFERING | FILE_FLAG_OVERLAPPED, NULL);

И работать уже с переменными внутри функции потока - то данные структуры разрушаются, а переменные остаются.

Может я туплю и так должно быть? Или пора заново книжки читать по WinAPI и MSDN старенький от MSVC++ 6.0 перечитывать?

@ISergey · 07.09.2019, 10:23

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

После вызова данной функции данные структуры m_args изменяются на черте что

как создаете поток и передаете структуру?

К примеру

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
struct ARGS
{
    TCHAR pszFileSrc[MAX_PATH];
    TCHAR pszFileDst[MAX_PATH];
    HWND m_test;
    HWND m_Main;
};
 
 
 
int main(){
 
    ...
    
    {
        ARGS arg = {..};
        CreateThread(.., threadFunc, &arg, ..);
        
    }
    
    // Здесь arg уже не существует (тоесть на стеке может быть уже затерто)
}

Должно повторить вашу ошибку.

Попробуйте так

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
int main(){
 
    ...
    
    {
        ARGS *arg = new ARGS;
        //заполнение arg 
        ....
        CreateThread(.., threadFunc, arg, ..);
        
    }
    
    // arg существует пока сами не удалим
}
 
DWORD WINAPI threadFunc(LPVOID lpParam)
{
    ARGS* arg = (ARGS*) lpParam;
    
    ...
    
    detele arg;
}

@ids1488 · 07.09.2019, 12:41 **[ТС]**

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

Должно повторить вашу ошибку.

Так и делаю. Вы правы.

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

Попробуйте так

Попробовал. На Windows 7 (там где и будет работать программа) все корректно. А на Windows 10 (Version 10.0.18362.267) в отладчике VS та же ситуация.

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

detele arg;

А разве при вызове из потока ExitThread(-1) не уничтожаются все данные потока? Или лучше самому явно удалить?

@ISergey · 07.09.2019, 12:54

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

А разве при вызове из потока ExitThread(-1) не уничтожаются все данные потока? Или лучше самому явно удалить?

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

Все что выделяется через new/malloc должно быть освобождено вами когда больше не нужно функциями delete/free

А на Windows 10 (Version 10.0.18362.267) в отладчике VS та же ситуация.

показывайте как создается поток и подготавливаются даные перед запуском потока.

@ids1488 · 07.09.2019, 13:03 **[ТС]**

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

показывайте как создается поток и подготавливаются даные перед запуском потока.

C++
1
2
3
4
5
6
7
8
ARGS *m_args = new ARGS;
_stprintf(m_args->pszFileSrc, _T("%s"), szPathname);
 _stprintf(m_args->pszFileDst, _T("%s"), szSavePathname);
m_args->m_test = GetDlgItem(hWnd, IDC_PROGRESS1);
m_args->m_Main = hWnd;
 
DWORD dwThreadID;
g_nThread = CreateThread(0,0,(LPTHREAD_START_ROUTINE)FileCopy, &m_args,0, &dwThreadID);

И на delete вылетает в Exception.

@ISergey · 07.09.2019, 14:55

Здесь

C++
1
_nThread = CreateThread(0,0,(LPTHREAD_START_ROUTINE)FileCopy, &m_args,0, &dwThreadID)

Перед m_args лишний &.

@ids1488 · 07.09.2019, 15:58 **[ТС]**

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

Перед m_args лишний &.

Это то понятно. Скопировал из cpp из блокнота, а не из VS.

Но все равно вот во что превращаются элементы структуры, когда вызывается первый CreateFile(...):

pszFileSrc = 0x07411208 L"ĥ罨ܾDB\\adb.exe"
pszFileDst = 0x07411410 L"C:\\DOCS\\adb.exe"
m_test = 0x000709aa {unused=??? }
m_Main = 0x000709e4 {unused=??? }

А значения до вызова (сразу по старте потока):

(m_args)->pszFileSrc 0x07411208 L"C:\\ADB\\adb.exe"
(m_args)->pszFileDst 0x07411410 L"C:\\DOCS\\adb.exe._FULL.bin"
m_args->m_test = 0x000709aa {unused=??? }
m_args->m_Main = 0x000709e4 {unused=??? }

Интересная закономерность: в Windows 7 + MSVS 6.0 все норм. В Windows 10 + MSVS 2015 Pro такая фигня. Уже не знаю на что грешить. Толи на винду с торрента, то ли на MSVS оттуда же.

По идее же CreateFile() не вносит изменения в переменную с именем файла. Не должен по крайней мере.

@ids1488 · 08.09.2019, 20:29 **[ТС]**

Все. Разобрался.

Выкинул заголовок с переопределениями функций и макросов от Рихтера и реализовал все штатными средствами Win API. Все корректно работает.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@ids1488 0 / 0 / 0 Регистрация: 07.09.2019 Сообщений: 5
	08.09.2019, 20:29 [ТС]
	Все. Разобрался. Выкинул заголовок с переопределениями функций и макросов от Рихтера и реализовал все штатными средствами Win API. Все корректно работает. 0