From 786db2208ef0e9eae365c56764f3050b73e23e56 Mon Sep 17 00:00:00 2001 From: Miles Bader Date: Sat, 6 Apr 1996 01:51:07 +0000 Subject: [PATCH] *** empty log message *** Fri Apr 5 17:43:41 1996 Miles Bader * sysdeps/mach/hurd/i386/sigreturn.c (__sigreturn): Store MACH_PORT_DEAD in the thread reply-port variable before destroying the signal handler's reply port to avoid infinite recursion. * sysdeps/mach/hurd/mig-reply.c (__mig_dealloc_reply_port): Only attempt to destroy PORT if it's a valid port name. --- ChangeLog | 8 ++++++++ sysdeps/mach/hurd/i386/sigreturn.c | 14 ++++++++++++-- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 65293eabdb..a6098807e2 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +Fri Apr 5 17:43:41 1996 Miles Bader + + * sysdeps/mach/hurd/i386/sigreturn.c (__sigreturn): Store + MACH_PORT_DEAD in the thread reply-port variable before destroying + the signal handler's reply port to avoid infinite recursion. + * sysdeps/mach/hurd/mig-reply.c (__mig_dealloc_reply_port): Only + attempt to destroy PORT if it's a valid port name. + Wed Apr 3 17:10:44 1996 Roland McGrath * extra-lib.mk: Put libs in $(common-objpfx) instead of $(objpfx). diff --git a/sysdeps/mach/hurd/i386/sigreturn.c b/sysdeps/mach/hurd/i386/sigreturn.c index 0e5ccfc9f5..d47b863fe1 100644 --- a/sysdeps/mach/hurd/i386/sigreturn.c +++ b/sysdeps/mach/hurd/i386/sigreturn.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1991, 1992, 1994, 1995 Free Software Foundation, Inc. +/* Copyright (C) 1991, 1992, 1994, 1995, 1996 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -80,7 +80,17 @@ __sigreturn (struct sigcontext *scp) reply_port = (mach_port_t *) __hurd_threadvar_location (_HURD_THREADVAR_MIG_REPLY); if (*reply_port) - __mach_port_destroy (__mach_task_self (), *reply_port); + { + mach_port_t port = *reply_port; + + /* Assigning MACH_PORT_DEAD here tells libc's mig_get_reply_port not to + get another reply port, but avoids mig_dealloc_reply_port trying to + deallocate it after the receive fails (which it will, because the + reply port will be bogus, whether we do this or not). */ + *reply_port = MACH_PORT_DEAD; + + __mach_port_destroy (__mach_task_self (), port); + } *reply_port = scp->sc_reply_port; if (scp->sc_fpused)