Do not evaluate this->next after calling the handler; the handler may

have clobbered it. Resolves core dumps of cvs server on user ^C.

ok millert@ deraadt@

1 files changed, 3 insertions, 1 deletions

diff --git a/gnu/usr.bin/cvs/lib/sighandle.c b/gnu/usr.bin/cvs/lib/sighandle.c
index f595dd24912..45b7477d014 100644
--- a/gnu/usr.bin/cvs/lib/sighandle.c
+++ b/gnu/usr.bin/cvs/lib/sighandle.c
@@ -155,8 +155,10 @@ int			sig;
 	this = SIG_handlers[sig];
 	while (this != (struct SIG_hlist *) NULL)
 	{
-		(*this->handler)(sig);
+		/* handler may free this (and thus clobber this->next) */
+		struct SIG_hlist *current = this;
 		this = this->next;
+		(*current->handler)(sig);
 	}
 
 	return;