From owner-p4-projects@FreeBSD.ORG  Thu Feb 10 17:19:39 2011
Return-Path: <owner-p4-projects@FreeBSD.ORG>
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id F42321065672; Thu, 10 Feb 2011 17:19:38 +0000 (UTC)
Delivered-To: perforce@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id B6BE9106566C
	for <perforce@freebsd.org>; Thu, 10 Feb 2011 17:19:38 +0000 (UTC)
	(envelope-from trasz@freebsd.org)
Received: from skunkworks.freebsd.org (skunkworks.freebsd.org
	[IPv6:2001:4f8:fff6::2d])
	by mx1.freebsd.org (Postfix) with ESMTP id A40A68FC0A
	for <perforce@freebsd.org>; Thu, 10 Feb 2011 17:19:38 +0000 (UTC)
Received: from skunkworks.freebsd.org (localhost [127.0.0.1])
	by skunkworks.freebsd.org (8.14.4/8.14.4) with ESMTP id p1AHJcOS002358
	for <perforce@freebsd.org>; Thu, 10 Feb 2011 17:19:38 GMT
	(envelope-from trasz@freebsd.org)
Received: (from perforce@localhost)
	by skunkworks.freebsd.org (8.14.4/8.14.4/Submit) id p1AHJc9q002355
	for perforce@freebsd.org; Thu, 10 Feb 2011 17:19:38 GMT
	(envelope-from trasz@freebsd.org)
Date: Thu, 10 Feb 2011 17:19:38 GMT
Message-Id: <201102101719.p1AHJc9q002355@skunkworks.freebsd.org>
X-Authentication-Warning: skunkworks.freebsd.org: perforce set sender to
	trasz@freebsd.org using -f
From: Edward Tomasz Napierala <trasz@FreeBSD.org>
To: Perforce Change Reviews <perforce@freebsd.org>
Precedence: bulk
Cc: 
Subject: PERFORCE change 188689 for review
X-BeenThere: p4-projects@freebsd.org
X-Mailman-Version: 2.1.5
List-Id: p4 projects tree changes <p4-projects.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/p4-projects>
List-Post: <mailto:p4-projects@freebsd.org>
List-Help: <mailto:p4-projects-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 10 Feb 2011 17:19:39 -0000

http://p4web.freebsd.org/@@188689?ac=10

Change 188689 by trasz@trasz_victim on 2011/02/10 17:19:24

	Replace signal(3) with sigaction(3).

Affected files ...

.. //depot/projects/soc2009/trasz_limits/lib/libutil/login_class.c#8 edit

Differences ...

==== //depot/projects/soc2009/trasz_limits/lib/libutil/login_class.c#8 (text+ko) ====

@@ -40,6 +40,7 @@
 #include <login_cap.h>
 #include <paths.h>
 #include <pwd.h>
+#include <signal.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -425,7 +426,7 @@
     quad_t	p;
     mode_t	mymask;
     login_cap_t *llc = NULL;
-    sig_t	prevsig;
+    struct sigaction sa, prevsa;
     struct rtprio rtp;
     int error;
 
@@ -519,9 +520,12 @@
 	 * XXX: This is a workaround to fail gracefully in case the kernel
 	 *      does not support setloginclass(2).
 	 */
-	prevsig = signal(SIGSYS, SIG_IGN);
+	bzero(&sa, sizeof(sa));
+	sa.sa_handler = SIG_IGN;
+	sigfillset(&sa.sa_mask);
+	sigaction(SIGSYS, &sa, &prevsa);
 	error = setloginclass(lc->lc_class);
-	signal(SIGSYS, prevsig);
+	sigaction(SIGSYS, &prevsa, NULL);
         if (error != 0) {
 	    syslog(LOG_ERR, "setloginclass(%s): %m", lc->lc_class);
 #ifdef notyet