Repository navigation
return in finally swallows exceptions #294
Description
Activity
Hi @iritkatriel, thank you for the issue report... now, this will likely not be fixed in pydevd because that code isn't really used...
At this point just a teeny-tiny bit of
winappdbgis actually used inpydevd(some win32 wrapping in the attach to process to inject a dll into the target process), and I think 99% of the code could be removed -- at this point it'd probably make sense to just port over the needed definitions and remove everything else, but this takes time and ends up being very much low priority vs other things, so, not sure if it'll ever be done -- I accept PRs though ;)
In
PyDev.Debugger/pydevd_attach_to_process/winappdbg/event.py
Line 1478 in 7d6e6e6
and
PyDev.Debugger/pydevd_attach_to_process/winappdbg/event.py
Line 1635 in 7d6e6e6
there is a
returnstatement in afinallyblock, which would swallow any in-flight exception.This means that if any exception (including
BaseExceptionsuch asKeyboardInterrupt) is raised from thetrybody, it will not propagate on as expected.See also https://docs.python.org/3/tutorial/errors.html#defining-clean-up-actions.