Page MenuHomeFreeBSD

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

Authored by emaste on Aug 25 2016, 6:47 PM.
Tags
None
Referenced Files
F80131902: D7644.id19692.diff
Thu, Mar 28, 8:18 AM
Unknown Object (File)
Wed, Mar 6, 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
Unknown Object (File)
Feb 1 2024, 2:17 PM
Unknown Object (File)
Jan 16 2024, 1:31 AM
Unknown Object (File)
Dec 22 2023, 10:52 PM
Subscribers
None

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

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.

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

usr.bin/bsdiff/bspatch/bspatch.c
156

bonus whitespace fix

Some comments.

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

Blank space needed.

133

This can fail right?

134

dirname, can return NULL.

137

This can fail.

usr.bin/bsdiff/bspatch/bspatch.c
136

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

usr.bin/bsdiff/bspatch/bspatch.c
136

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

oshogbo edited edge metadata.

Thank you for doing this.

This revision is now accepted and ready to land.Aug 25 2016, 8:48 PM
This revision was automatically updated to reflect the committed changes.