upstream: Replace all calls to signal(2) with a wrapper around

sigaction(2). This wrapper blocks all other signals during the handler
preventing races between handlers, and sets SA_RESTART which should reduce
the potential for short read/write operations.

OpenBSD-Commit-ID: 5e047663fd77a40d7b07bdabe68529df51fd2519

1 files changed, 2 insertions, 2 deletions

diff --git a/openbsd-compat/bsd-openpty.c b/openbsd-compat/bsd-openpty.c
index 123a9be56..b6b5ab49b 100644
--- a/openbsd-compat/bsd-openpty.c
+++ b/openbsd-compat/bsd-openpty.c
@@ -103,10 +103,10 @@ openpty(int *amaster, int *aslave, char *name, struct termios *termp,
                 return (-1);
 
         /* XXX: need to close ptm on error? */
-        old_signal = signal(SIGCHLD, SIG_DFL);
+        old_signal = ssh_signal(SIGCHLD, SIG_DFL);
         if (grantpt(ptm) < 0)
                 return (-1);
-        signal(SIGCHLD, old_signal);
+        ssh_signal(SIGCHLD, old_signal);
 
         if (unlockpt(ptm) < 0)
                 return (-1);