From owner-freebsd-bugs  Tue Apr 16 05:47:42 1996
Return-Path: owner-bugs
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id FAA08654
          for bugs-outgoing; Tue, 16 Apr 1996 05:47:42 -0700 (PDT)
Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.2.228.19])
          by freefall.freebsd.org (8.7.3/8.7.3) with SMTP id FAA08649
          for <freebsd-bugs@freefall.freebsd.org>; Tue, 16 Apr 1996 05:47:38 -0700 (PDT)
Received: (from bde@localhost) by godzilla.zeta.org.au (8.6.12/8.6.9) id WAA32001; Tue, 16 Apr 1996 22:37:47 +1000
Date: Tue, 16 Apr 1996 22:37:47 +1000
From: Bruce Evans <bde@zeta.org.au>
Message-Id: <199604161237.WAA32001@godzilla.zeta.org.au>
To: bde@zeta.org.au, fenner@parc.xerox.com, freebsd-bugs@freefall.freebsd.org
Subject: Re:  kern/1144: sig{add, del}set and sigismember fns don't check signo
Sender: owner-bugs@FreeBSD.ORG
X-Loop: FreeBSD.org
Precedence: bulk

I wrote:

>#define sigaddset(set, signo)					\
>	({ struct sigaction *__set = set;			\
>	   int __signo = (signo);				\
>	   int __rv;						\
>								\
>	   /* 32 because NSIG is in application namespace. */	\
>	   if (__signo <= 0 || __signo >= 32) {			\
>	   	errno = EINVAL;					\

Namespace stuff is tricky.  I think EINVAL isn't supposed to be
visible if only <signal.h> is included, so it can't be used
directly.

Bruce