diff options
| author |   dtucker <dtucker@openbsd.org> | 2011-06-03 01:37:40 +0000 |
|---|---|---|
| committer | dtucker <dtucker@openbsd.org> | 2011-06-03 01:37:40 +0000 |
| commit | 93534fb9d638a76d03a257e90efc1133068cec15 (patch) | |
| tree | 347d286236b1e402eca911a30e8d1e9149c4c756 /usr.bin/ssh/ssh-agent.c | |
| parent | bz#1883 - setproctitle() to identify mux master; patch from Bert.Wesarg (diff) | |
| download | wireguard-openbsd-93534fb9d638a76d03a257e90efc1133068cec15.tar.xz wireguard-openbsd-93534fb9d638a76d03a257e90efc1133068cec15.zip | |
Check current parent process ID against saved one to determine if the parent
has exited, rather than attempting to send a zero signal, since the latter
won't work if the parent has changed privs. bz#1905, patch from Daniel Kahn
Gillmor, ok djm@
Diffstat (limited to 'usr.bin/ssh/ssh-agent.c')
| -rw-r--r-- | usr.bin/ssh/ssh-agent.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/usr.bin/ssh/ssh-agent.c b/usr.bin/ssh/ssh-agent.c index 4768c9c3cc2..ca962f1094b 100644 --- a/usr.bin/ssh/ssh-agent.c +++ b/usr.bin/ssh/ssh-agent.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ssh-agent.c,v 1.171 2010/11/21 01:01:13 djm Exp $ */ +/* $OpenBSD: ssh-agent.c,v 1.172 2011/06/03 01:37:40 dtucker Exp $ */ /* * Author: Tatu Ylonen <ylo@cs.hut.fi> * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland @@ -1076,7 +1076,11 @@ cleanup_handler(int sig) static void check_parent_exists(void) { - if (parent_pid != -1 && kill(parent_pid, 0) < 0) { + /* + * If our parent has exited then getppid() will return (pid_t)1, + * so testing for that should be safe. + */ + if (parent_pid != -1 && getppid() != parent_pid) { /* printf("Parent has died - Authentication agent exiting.\n"); */ cleanup_socket(); _exit(2); |