* gendef (sigbe): Reset "incyg" while the stack lock is active to avoid a

potential race.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 5fca04d..0e08931 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,8 @@
+2007-08-09  Ernie Coskrey  <Ernie.Coskrey@steeleye.com>
+
+       * gendef (sigbe): Reset "incyg" while the stack lock is active to avoid
+       a potential race.
+
 2007-08-09  Corinna Vinschen  <corinna@vinschen.de>
 
        * syscalls.cc (rename): Close oldpath file handle immediately after

diff --git a/winsup/cygwin/gendef b/winsup/cygwin/gendef
index 4f60ea6..0d7df4b 100755 (executable)
--- a/winsup/cygwin/gendef
+++ b/winsup/cygwin/gendef
@@ -158,9 +158,9 @@ __sigbe:                                            # return here after cygwin syscall
        xorl    %edx,%edx
        xchgl   %edx,-4(%eax)                           # get return address from signal stack
        xchgl   %edx,8(%esp)                            # restore edx/real return address
+       decl    $tls::incyg(%ebx)
        decl    $tls::stacklock(%ebx)                   # release lock
        popl    %eax
-       decl    $tls::incyg(%ebx)
        popl    %ebx
        ret