diff options
author | Brian Gianforcaro <bgianf@serenityos.org> | 2021-06-06 16:34:03 -0700 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-06-07 09:43:30 +0200 |
commit | 77f4f6e0dea39461d5a806f376455546df4e727d (patch) | |
tree | 4922d8456b7d95fbfb4262aadadc42011f5c0ed9 /Toolchain | |
parent | 9fccbde371cf06334c335469a01a153454c4596a (diff) | |
download | serenity-77f4f6e0dea39461d5a806f376455546df4e727d.zip |
Kernel: Fix error propagation if Thread::WaitBlocker constructor fails
There is logic at the end of the constructor that sets m_should_block
to false if we encountered errors. We were missing this step due to the
erroneous early return, the code then ended up waiting and then
asserting on unblock since the WaitBlocker is in a invalid state.
This fix is to not return early, and let normal control flow handle it.
Fixes: #7857
Verified with `stress-ng --yield=10` locally.
Diffstat (limited to 'Toolchain')
0 files changed, 0 insertions, 0 deletions