diff options
| author | Gregory P. Smith <greg@krypto.org> | 2012-11-11 04:33:39 (GMT) |
|---|---|---|
| committer | Gregory P. Smith <greg@krypto.org> | 2012-11-11 04:33:39 (GMT) |
| commit | 34b14951eeb235ac0e0cbbd2ccd5d72d178dd983 (patch) | |
| tree | 89af57c899c0ed161adbcc0fe3374d68d1ce9f69 | |
| parent | ec6dfcffa05414e7ee29cfe88551a3d3dcdaafdc (diff) | |
| parent | 9463e3ac8b230efa2f35e08859cb3db5c6d192b7 (diff) | |
| download | cpython-34b14951eeb235ac0e0cbbd2ccd5d72d178dd983.zip cpython-34b14951eeb235ac0e0cbbd2ccd5d72d178dd983.tar.gz cpython-34b14951eeb235ac0e0cbbd2ccd5d72d178dd983.tar.bz2 | |
Fixes issue #9535: Fix pending signals that have been received but not
yet handled by Python to not persist after os.fork() in the child process.
| -rw-r--r-- | Misc/NEWS | 3 | ||||
| -rw-r--r-- | Modules/signalmodule.c | 16 |
2 files changed, 19 insertions, 0 deletions
@@ -12,6 +12,9 @@ What's New in Python 3.3.1? Core and Builtins ----------------- +- Issue #9535: Fix pending signals that have been received but not yet + handled by Python to not persist after os.fork() in the child process. + - Issue #15001: fix segfault on "del sys.module['__main__']". Patch by Victor Stinner. diff --git a/Modules/signalmodule.c b/Modules/signalmodule.c index 2eb7f29..eb89a03 100644 --- a/Modules/signalmodule.c +++ b/Modules/signalmodule.c @@ -1394,9 +1394,25 @@ PyOS_InterruptOccurred(void) return 0; } +static void +_clear_pending_signals(void) +{ + int i; + if (!is_tripped) + return; + is_tripped = 0; + for (i = 1; i < NSIG; ++i) { + Handlers[i].tripped = 0; + } +} + void PyOS_AfterFork(void) { + /* Clear the signal flags after forking so that they aren't handled + * in both processes if they came in just before the fork() but before + * the interpreter had an opportunity to call the handlers. issue9535. */ + _clear_pending_signals(); #ifdef WITH_THREAD /* PyThread_ReInitTLS() must be called early, to make sure that the TLS API * can be called safely. */ |