diff options
| author | Corinna Vinschen <corinna@vinschen.de> | 2015-11-03 18:25:23 +0100 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2015-11-03 18:28:29 +0100 |
| commit | 179415a9d7d473bbba17744abc7769db0f6462e8 (patch) | |
| tree | ee74557ed22066be634eeb0ea5d58ea6b80ee9ee | |
| parent | 8631a3bb8d6d48e05e06db6c936fc6142c4945cf (diff) | |
| download | newlib-179415a9d7d473bbba17744abc7769db0f6462e8.zip newlib-179415a9d7d473bbba17744abc7769db0f6462e8.tar.gz newlib-179415a9d7d473bbba17744abc7769db0f6462e8.tar.bz2 | |
Fix potential endless loop in pending_signals::clear
* sigproc.cc (pending_signals::clear): Fix previous fix resulting in
yet another endless loop.
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
| -rw-r--r-- | winsup/cygwin/ChangeLog | 5 | ||||
| -rw-r--r-- | winsup/cygwin/sigproc.cc | 13 |
2 files changed, 9 insertions, 9 deletions
diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index e7dc642..b6e7255 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,8 @@ +2015-11-03 Corinna Vinschen <corinna@vinschen.de> + + * sigproc.cc (pending_signals::clear): Fix previous fix resulting in + yet another endless loop. + 2015-11-02 Corinna Vinschen <corinna@vinschen.de> * include/netinet/ip.h (MAX_IPOPTLEN): Define. diff --git a/winsup/cygwin/sigproc.cc b/winsup/cygwin/sigproc.cc index fbc738d..6a7708f 100644 --- a/winsup/cygwin/sigproc.cc +++ b/winsup/cygwin/sigproc.cc @@ -402,16 +402,11 @@ sig_clear (int sig) void pending_signals::clear (_cygtls *tls) { - sigpacket *q = &start, *qnext; + sigpacket *q, *qnext; - while ((qnext = q->next)) - { - if (qnext->sigtls == tls) - { - q->next = qnext->next; - qnext->si.si_signo = 0; - } - } + for (q = &start; (qnext = q->next); q->next = qnext->next) + if (qnext->sigtls == tls) + qnext->si.si_signo = 0; } /* Clear pending signals of specific thread. Called from _cygtls::remove */ |