Date: Sun, 18 Jul 2010 22:10:08 -0700 From: Garrett Cooper <yanegomi@gmail.com> To: hackers@freebsd.org Cc: Kostik Belousov <kostikbel@gmail.com> Subject: Fwd: [PATCH] Catch errors with sigaddset(3) in sigaddset (*sigset) Message-ID: <AANLkTimiaHZ_cBykjPXr2tev3vBTHN-uO-riHepBrEi4@mail.gmail.com> In-Reply-To: <AANLkTilPpmzhb39E-R2ni3p2l4KR1kaeNS-EZJvbK6fE@mail.gmail.com> References: <AANLkTilPpmzhb39E-R2ni3p2l4KR1kaeNS-EZJvbK6fE@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
CCing hackers@. ---------- Forwarded message ---------- From: Garrett Cooper <yanegomi@gmail.com> Date: Sun, Jul 18, 2010 at 10:06 PM Subject: [PATCH] Catch errors with sigaddset(3) in sigaddset (*sigset) To: Kostik Belousov <kostikbel@gmail.com> None of the sigprocmask(2) code actually checks to see whether or not the signal set is valid. This fixes that. Thanks, -Garrett Index: compat-43/sigcompat.c =================================================================== --- compat-43/sigcompat.c (revision 210226) +++ compat-43/sigcompat.c (working copy) @@ -163,7 +163,9 @@ int error; sigemptyset(&set); - sigaddset(&set, sig); + error = sigaddset(&set, sig); + if (error == -1) + return (SIG_ERR); error = _sigprocmask(SIG_BLOCK, NULL, &pset); if (error == -1) return (SIG_ERR);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTimiaHZ_cBykjPXr2tev3vBTHN-uO-riHepBrEi4>