From 75d12dffe707d6861485092e0a87db6384a87252 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexander=20F=C3=A6r=C3=B8y?= Date: Tue, 1 Nov 2022 10:45:17 +0100 Subject: [PATCH] Remove `BUG()` in Windows process read callback. This BUG() was added when the code was written to see if this callback was ever executed after we marked the handle as EOF. It turns out, it does, but we handle it gracefully. We can therefore remove the BUG(). Fixes tpo/core/tor#40596. --- changes/ticket40596 | 4 ++++ src/lib/process/process_win32.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) create mode 100644 changes/ticket40596 diff --git a/changes/ticket40596 b/changes/ticket40596 new file mode 100644 index 0000000000..13c8e5b34a --- /dev/null +++ b/changes/ticket40596 @@ -0,0 +1,4 @@ + o Minor bugfixes (pluggable transports, windows): + - Remove a warning `BUG()` that could occur when attempting to execute a + non-existing pluggable transport on Windows. Fixes bug 40596; bugfix on + 0.4.0.1-alpha. diff --git a/src/lib/process/process_win32.c b/src/lib/process/process_win32.c index dfcb17a480..6458f93752 100644 --- a/src/lib/process/process_win32.c +++ b/src/lib/process/process_win32.c @@ -888,7 +888,7 @@ process_win32_read_from_handle(process_win32_handle_t *handle, /* Check if we have been asked to read from a handle that have already told * us that we have reached the end of the file. */ - if (BUG(handle->reached_eof)) + if (handle->reached_eof) return 0; /* This cast should be safe since our buffer can be at maximum up to