NetBSD Problem Report #45878

From www@NetBSD.org  Fri Jan 27 09:23:37 2012
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [204.152.190.11])
	by www.NetBSD.org (Postfix) with ESMTP id 8893363DE10
	for <gnats-bugs@gnats.NetBSD.org>; Fri, 27 Jan 2012 09:23:37 +0000 (UTC)
Message-Id: <20120127092336.E6AD563D5E6@www.NetBSD.org>
Date: Fri, 27 Jan 2012 09:23:36 +0000 (UTC)
From: skrll@netbsd.org
Reply-To: skrll@netbsd.org
To: gnats-bugs@NetBSD.org
Subject: SIG_ATOMIC_{MAX,MIN} wrong for sig_atomic_t on amd64
X-Send-Pr-Version: www-1.0

>Number:         45878
>Category:       port-amd64
>Synopsis:       SIG_ATOMIC_{MAX,MIN} wrong for sig_atomic_t on amd64
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    port-amd64-maintainer
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Jan 27 09:25:00 +0000 2012
>Closed-Date:    Fri Jan 27 16:19:37 +0000 2012
>Last-Modified:  Fri Jan 27 16:19:37 +0000 2012
>Originator:     Nick Hudson
>Release:        -current
>Organization:
n/a
>Environment:
n/a
>Description:
kenny$ grep typedef.*sig_atomic_t sys/arch/amd64/include/*          
sys/arch/amd64/include/signal.h:typedef int sig_atomic_t;
kenny$ grep SIG_ATOMIC_ sys/arch/amd64/include/*           
sys/arch/amd64/include/int_limits.h:#define     SIG_ATOMIC_MIN  (-0x7fffffffffffffffL-1)        /* sig_atomic_t   */
sys/arch/amd64/include/int_limits.h:#define     SIG_ATOMIC_MAX  0x7fffffffffffffffL             /* sig_atomic_t   */
kenny$ 

>How-To-Repeat:
Code inspection
>Fix:
not provided

>Release-Note:

>Audit-Trail:
From: "Christos Zoulas" <christos@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/45878 CVS commit: src/sys/arch/amd64/include
Date: Fri, 27 Jan 2012 10:41:02 -0500

 Module Name:	src
 Committed By:	christos
 Date:		Fri Jan 27 15:41:02 UTC 2012

 Modified Files:
 	src/sys/arch/amd64/include: int_limits.h

 Log Message:
 PR/45878: Nick Hudson: SIG_ATOMIC_{MAX,MIN} wrong for sig_atomic_t on amd64
 sig_atomic_t is an int on amd64, put the proper limits there


 To generate a diff of this commit:
 cvs rdiff -u -r1.7 -r1.8 src/sys/arch/amd64/include/int_limits.h

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->closed
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Fri, 27 Jan 2012 16:19:37 +0000
State-Changed-Why:
christos fixed it


>Unformatted:

Home	PR Database Search