PR/33753 CVS commit: src/bin/pax

["Luke Mewburn" <lukem%netbsd.org@localhost>]

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

From: "Luke Mewburn" <lukem%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/33753 CVS commit: src/bin/pax
Date: Sun, 28 May 2023 21:42:40 +0000

 Module Name:	src
 Committed By:	lukem
 Date:		Sun May 28 21:42:40 UTC 2023
 
 Modified Files:
 	src/bin/pax: ar_subs.c buf_subs.c extern.h file_subs.c
 
 Log Message:
 pax: don't overwrite destination if -r -w copy fails
 
 Add more error handling to pax -r -w so that any failure
 during the copy to the temporary file (including a failed flush)
 prevents any existing destination file from being replaced
 with the partial (including possibly empty) temporary file.
 The partial temporary file is removed.  pax still exists non-zero.
 
 Thanks to Michael van Elst (mlelstv@) for the analysis
 of the problem in the PR.
 
 Should fix PR misc/33753.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.57 -r1.58 src/bin/pax/ar_subs.c
 cvs rdiff -u -r1.30 -r1.31 src/bin/pax/buf_subs.c
 cvs rdiff -u -r1.60 -r1.61 src/bin/pax/extern.h
 cvs rdiff -u -r1.64 -r1.65 src/bin/pax/file_subs.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.