diff options
| author |   dtucker <dtucker@openbsd.org> | 2018-02-11 21:16:56 +0000 |
|---|---|---|
| committer | dtucker <dtucker@openbsd.org> | 2018-02-11 21:16:56 +0000 |
| commit | 6197b654837c0ab7e19ac6a1e307fdb73ec6b9f9 (patch) | |
| tree | fdb8854fdae23e0b9edcb27ca7b5b79ec2703a76 /usr.bin/ssh/serverloop.c | |
| parent | Use the new APIs for setting block lengths and reading from/writing to (diff) | |
| download | wireguard-openbsd-6197b654837c0ab7e19ac6a1e307fdb73ec6b9f9.tar.xz wireguard-openbsd-6197b654837c0ab7e19ac6a1e307fdb73ec6b9f9.zip | |
Don't reset signal handlers inside handlers.
The signal handlers from the original ssh1 code on which OpenSSH
is based assume unreliable signals and reinstall their handlers.
Since OpenBSD (and pretty much every current system) has reliable
signals this is not needed. In the unlikely even that -portable
is still being used on such systems we will deal with it in the
compat layer. ok deraadt@
Diffstat (limited to 'usr.bin/ssh/serverloop.c')
| -rw-r--r-- | usr.bin/ssh/serverloop.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/usr.bin/ssh/serverloop.c b/usr.bin/ssh/serverloop.c index dd213cd4237..70436e44335 100644 --- a/usr.bin/ssh/serverloop.c +++ b/usr.bin/ssh/serverloop.c @@ -1,4 +1,4 @@ -/* $OpenBSD: serverloop.c,v 1.203 2018/01/23 05:27:21 djm Exp $ */ +/* $OpenBSD: serverloop.c,v 1.204 2018/02/11 21:16:56 dtucker Exp $ */ /* * Author: Tatu Ylonen <ylo@cs.hut.fi> * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland @@ -149,7 +149,6 @@ sigchld_handler(int sig) { int save_errno = errno; child_terminated = 1; - signal(SIGCHLD, sigchld_handler); notify_parent(); errno = save_errno; } |