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
Unknown Object (File)
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

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
153 ↗(On Diff #19686)

bonus whitespace fix

Some comments.

usr.bin/bsdiff/bspatch/bspatch.c
58 ↗(On Diff #19686)

Blank space needed.

132 ↗(On Diff #19686)

This can fail right?

133 ↗(On Diff #19686)

dirname, can return NULL.

136 ↗(On Diff #19686)

This can fail.

usr.bin/bsdiff/bspatch/bspatch.c
136 ↗(On Diff #19690)

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

usr.bin/bsdiff/bspatch/bspatch.c
136 ↗(On Diff #19690)

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.