Page MenuHomeFreeBSD
Differential D13402

Don't free() uninitialized memory
ClosedPublic
Actions

Authored by markj on Dec 7 2017, 12:01 AM.
Tags
None
Referenced Files
F171126971: D13402.id36339.diff
Tue, Sep 8, 9:12 PM
F171126184: D13402.id36325.diff
Tue, Sep 8, 9:06 PM
F171125730: D13402.diff
Tue, Sep 8, 9:03 PM
Unknown Object (File)
Sun, Sep 6, 2:42 AM
Unknown Object (File)
Tue, Aug 25, 8:37 PM
Unknown Object (File)
Tue, Aug 25, 8:34 PM
Unknown Object (File)
Tue, Aug 25, 8:33 PM
Unknown Object (File)
Thu, Aug 20, 4:32 PM
View All Files
Subscribers
None

Details

Reviewers
imp
cem
Commits
rS332123: MFC efivar changes: r321429, r323056-r323057, r323066, r323259-r323260,
rS326658: Ensure that "out" is initialized in all error paths.
Summary

Caught by gcc.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 13397
Build 13629: arc lint + arc unit

Event Timeline

markj created this revision.Dec 7 2017, 12:01 AM
markj added a reviewer: imp.Dec 7 2017, 12:02 AM
cem accepted this revision.Dec 7 2017, 12:53 AM
This revision is now accepted and ready to land.Dec 7 2017, 12:53 AM
Closed by commit rS326658: Ensure that "out" is initialized in all error paths. (authored by markj). · Explain WhyDec 7 2017, 3:16 PM
This revision was automatically updated to reflect the committed changes.
imp added a comment.Dec 7 2017, 4:38 PM

Thanks for doing this, but please hold off any further changes to the efi stuff. I have it on a couple of branches on a couple of machines tweaking and even trivial changes like this cause me hassles in merging and updating. I'd prefer if you send me stuff like this directly. I don't even recall seeing this review.

Revision Contents
Changeset List

PathSize
lib/
libefivar/
2 lines

Diff 36325

View Options

lib/libefivar/efivar-dp-xlate.c

Loading...