From c714dfc936285fb588d047bb71b27cd396af6490 Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Thu, 1 Jul 2021 18:30:00 +0100 Subject: [PATCH] Close all thread handles returned from CreateThread. If you don't, they are permanently leaked. A user points out that this is particularly bad in Pageant, with the new named-pipe-based IPC, since it will spawn an input and output I/O thread per named pipe connection, leading to two handles being leaked every time. --- windows/handle-io.c | 12 ++++++++---- windows/pageant.c | 6 ++++-- windows/puttygen.c | 6 ++++-- windows/window.c | 6 ++++-- 4 files changed, 20 insertions(+), 10 deletions(-) diff --git a/windows/handle-io.c b/windows/handle-io.c index ecb48577..873eaf30 100644 --- a/windows/handle-io.c +++ b/windows/handle-io.c @@ -481,8 +481,10 @@ struct handle *handle_input_new(HANDLE handle, handle_inputfn_t gotdata, h->u.i.flags = flags; ensure_ready_event_setup(); - CreateThread(NULL, 0, handle_input_threadfunc, - &h->u.i, 0, &in_threadid); + HANDLE hThread = CreateThread(NULL, 0, handle_input_threadfunc, + &h->u.i, 0, &in_threadid); + if (hThread) + CloseHandle(hThread); /* we don't need the thread handle */ h->u.i.busy = true; return h; @@ -508,8 +510,10 @@ struct handle *handle_output_new(HANDLE handle, handle_outputfn_t sentdata, h->u.o.flags = flags; ensure_ready_event_setup(); - CreateThread(NULL, 0, handle_output_threadfunc, - &h->u.o, 0, &out_threadid); + HANDLE hThread = CreateThread(NULL, 0, handle_output_threadfunc, + &h->u.o, 0, &out_threadid); + if (hThread) + CloseHandle(hThread); /* we don't need the thread handle */ return h; } diff --git a/windows/pageant.c b/windows/pageant.c index c82c18bc..c1bc55cc 100644 --- a/windows/pageant.c +++ b/windows/pageant.c @@ -1631,8 +1631,10 @@ int WINAPI WinMain(HINSTANCE inst, HINSTANCE prev, LPSTR cmdline, int show) DWORD wm_copydata_threadid; wmct.ev_msg_ready = CreateEvent(NULL, false, false, NULL); wmct.ev_reply_ready = CreateEvent(NULL, false, false, NULL); - CreateThread(NULL, 0, wm_copydata_threadfunc, - &inst, 0, &wm_copydata_threadid); + HANDLE hThread = CreateThread(NULL, 0, wm_copydata_threadfunc, + &inst, 0, &wm_copydata_threadid); + if (hThread) + CloseHandle(hThread); /* we don't need the thread handle */ add_handle_wait(wmct.ev_msg_ready, wm_copydata_got_msg, NULL); if (show_keylist_on_startup) diff --git a/windows/puttygen.c b/windows/puttygen.c index cf98b6fe..7bc8f494 100644 --- a/windows/puttygen.c +++ b/windows/puttygen.c @@ -1160,13 +1160,15 @@ static void start_generating_key(HWND hwnd, struct MainDlgState *state) params->key = &state->key; params->dsakey = &state->dsakey; - if (!CreateThread(NULL, 0, generate_key_thread, - params, 0, &threadid)) { + HANDLE hThread = CreateThread(NULL, 0, generate_key_thread, + params, 0, &threadid); + if (!hThread) { MessageBox(hwnd, "Out of thread resources", "Key generation error", MB_OK | MB_ICONERROR); sfree(params); } else { + CloseHandle(hThread); /* we don't need the thread handle */ state->generation_thread_exists = true; } } diff --git a/windows/window.c b/windows/window.c index acfa3777..b00b910c 100644 --- a/windows/window.c +++ b/windows/window.c @@ -5355,8 +5355,10 @@ static void wintw_clip_request_paste(TermWin *tw, int clipboard) * that tells us it's OK to paste. */ DWORD in_threadid; /* required for Win9x */ - CreateThread(NULL, 0, clipboard_read_threadfunc, - wgs.term_hwnd, 0, &in_threadid); + HANDLE hThread = CreateThread(NULL, 0, clipboard_read_threadfunc, + wgs.term_hwnd, 0, &in_threadid); + if (hThread) + CloseHandle(hThread); /* we don't need the thread handle */ } /*