NetBSD Problem Report #47547

From www@NetBSD.org  Sat Feb  9 02:40:45 2013
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
	by www.NetBSD.org (Postfix) with ESMTP id 46D9563E520
	for <gnats-bugs@gnats.NetBSD.org>; Sat,  9 Feb 2013 02:40:45 +0000 (UTC)
Message-Id: <20130209024044.821F963E520@www.NetBSD.org>
Date: Sat,  9 Feb 2013 02:40:44 +0000 (UTC)
From: william@25thandclement.com
Reply-To: william@25thandclement.com
To: gnats-bugs@NetBSD.org
Subject: hexdump typo
X-Send-Pr-Version: www-1.0

>Number:         47547
>Category:       bin
>Synopsis:       hexdump typo
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    bin-bug-people
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Feb 09 02:45:00 +0000 2013
>Closed-Date:    Sat Feb 09 03:25:47 +0000 2013
>Last-Modified:  Sat Feb 09 03:25:47 +0000 2013
>Originator:     William Ahern
>Release:        NetBSD 5.1.2
>Organization:
>Environment:
NetBSD  5.1.2 NetBSD 5.1.2 (GENERIC) #0: Thu Feb  2 12:12:28 UTC 2012  builds@b7.netbsd.org:/home/builds/ab/netbsd-5-1-2-RELEASE/amd64/201202021012Z-obj/home/builds/ab/netbsd-5-1-2-RELEASE/src/sys/arch/amd64/compile/GENERIC amd64

>Description:
usr.bin/hexdump/conv.c contains a 20-year-old typo, near line 108 in trunk. The list[] array spells the ASCII label "dc1" as "dcl". Note the letter "l" versus number "1".

I've reported this bug to FreeBSD, OpenBSD, and Debian (which repackages BSD hexdump). Apple appears to have quietly fixed their copy in OS X.

I discovered this bug while testing my own implementation at
http://25thandclement.com/~william/projects/hexdump.c.html


>How-To-Repeat:
hexdump -e '16 "%3_u " "\n"' /dev/urandom | grep -e dcl -e dc1 | head
>Fix:
Fix typo.

>Release-Note:

>Audit-Trail:
From: "Jonathan A. Kollasch" <jakllsch@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/47547 CVS commit: src/usr.bin/hexdump
Date: Sat, 9 Feb 2013 02:49:36 +0000

 Module Name:	src
 Committed By:	jakllsch
 Date:		Sat Feb  9 02:49:36 UTC 2013

 Modified Files:
 	src/usr.bin/hexdump: conv.c

 Log Message:
 "dcl" is not "dc1".

 Fixes PR#47547


 To generate a diff of this commit:
 cvs rdiff -u -r1.13 -r1.14 src/usr.bin/hexdump/conv.c

 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: Sat, 09 Feb 2013 03:25:47 +0000
State-Changed-Why:
Fixed in HEAD, thanks.


>Unformatted:

Home	PR Database Search