PR/59942 CVS commit: [netbsd-11] src/bin/dd

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR bin/59942; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59942 CVS commit: [netbsd-11] src/bin/dd
Date: Mon, 2 Feb 2026 17:49:07 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Mon Feb  2 17:49:07 UTC 2026
 
 Modified Files:
 	src/bin/dd [netbsd-11]: args.c dd.1 dd.c
 
 Log Message:
 Pull up following revision(s) (requested by kre in ticket #171):
 
 	bin/dd/dd.1: revision 1.40
 	bin/dd/dd.1: revision 1.41
 	bin/dd/args.c: revision 1.44
 	bin/dd/dd.c: revision 1.54
 	bin/dd/dd.1: revision 1.39
 
 PR bin/59942 Fix handling of count=0
 
 Apply (modified) patch from Andrus Suvalau in PR bin/59942
 (The actual change is very simple)
 
 This looks to have been broken since version 1.1 - simply short
 circuiting processing because count==0 is wrong, there is much
 else that needs to be done before copying nothing (including
 verifying args that follow it).
 
 It is almost certain that files=0 will need a similar change.
 
 That one looks like it will be even easier - simply deleting
 the test for files_cnt being 0 in args.c/f_files() should be
 enough I believe - but as I haven't seen any system with a
 tape drive in decades, I am unable to test any changes properly,
 so I will leave that to someone who can test it properly.
 
 dd(1): touch up markup
 
 dd(1): clarify the text about oflag operand and creat as its default
 Hat tip to Ian D. Leroux for the suggested wording.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.43 -r1.43.4.1 src/bin/dd/args.c
 cvs rdiff -u -r1.38 -r1.38.6.1 src/bin/dd/dd.1
 cvs rdiff -u -r1.53 -r1.53.12.1 src/bin/dd/dd.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.