Page MenuHomeFreeBSD
Differential D7644

bspatch: clean up output file in the case of error
ClosedPublic
Actions

Authored by emaste on Aug 25 2016, 6:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 1:47 PM
Unknown Object (File)
Wed, Apr 17, 5:31 PM
Unknown Object (File)
Wed, Apr 17, 1:31 AM
Unknown Object (File)
Thu, Mar 28, 8:18 AM
Unknown Object (File)
Mar 6 2024, 5:33 AM
Unknown Object (File)
Feb 26 2024, 7:24 AM
Unknown Object (File)
Feb 11 2024, 11:55 PM
Unknown Object (File)
Feb 4 2024, 2:21 AM
View All 56 Files
Subscribers
None

Details

Reviewers
allanjude
delphij
cperciva
oshogbo
Commits
rS304821: bspatch: remove output file in the case of error

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste updated this revision to Diff 19683.Aug 25 2016, 6:47 PM
emaste retitled this revision from to bspatch: clean up output file in the case of error.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added reviewers: allanjude, oshogbo, cperciva, delphij.
emaste updated this revision to Diff 19686.Aug 25 2016, 7:14 PM

unlinkat is POSIX so no need to use it only in the #ifdef CAPSICUM case

usr.bin/bsdiff/bspatch/bspatch.c
153

bonus whitespace fix

oshogbo edited edge metadata.Aug 25 2016, 7:59 PM

Some comments.

usr.bin/bsdiff/bspatch/bspatch.c
58

Blank space needed.

132

This can fail right?

133

dirname, can return NULL.

136

This can fail.

emaste updated this revision to Diff 19690.Aug 25 2016, 8:35 PM
emaste edited edge metadata.

fixups from @oshogbo

oshogbo added inline comments.Aug 25 2016, 8:41 PM
usr.bin/bsdiff/bspatch/bspatch.c
135

namebuf can be NULL. Should we print argv[2]?

emaste updated this revision to Diff 19691.Aug 25 2016, 8:44 PM

add strdup failure check

emaste updated this revision to Diff 19692.Aug 25 2016, 8:47 PM

one more fix from @oshogbo

oshogbo added inline comments.Aug 25 2016, 8:47 PM
usr.bin/bsdiff/bspatch/bspatch.c
135

Sorry for being PITA but namebuf still can be NULL, in this err msg.

oshogbo accepted this revision.Aug 25 2016, 8:48 PM
oshogbo edited edge metadata.

Thank you for doing this.

This revision is now accepted and ready to land.Aug 25 2016, 8:48 PM
Closed by commit rS304821: bspatch: remove output file in the case of error (authored by emaste). · Explain WhyAug 25 2016, 9:33 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
usr.bin/
bsdiff/
bspatch/
30 lines

Diff 19686

View Options

usr.bin/bsdiff/bspatch/bspatch.c

Loading...