[prev in list] [next in list] [prev in thread] [next in thread]
List: netbsd-bugs
Subject: PR/33753 CVS commit: src/bin/pax
From: "Luke Mewburn" <lukem () netbsd ! org>
Date: 2023-05-28 21:45:01
Message-ID: 20230528214501.9CD051A923D () mollari ! NetBSD ! org
[Download RAW message or body]
The following reply was made to PR misc/33753; it has been noted by GNATS.
From: "Luke Mewburn" <lukem@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
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.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic