Page MenuHomeFreeBSD

imgact_elf: Ensure that the return value in parse_notes is initialized
ClosedPublic

Authored by markj on Apr 25 2021, 11:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 5 2024, 3:08 AM
Unknown Object (File)
Nov 7 2024, 3:21 PM
Unknown Object (File)
Oct 23 2024, 12:15 PM
Unknown Object (File)
Oct 12 2024, 7:29 AM
Unknown Object (File)
Oct 2 2024, 8:30 AM
Unknown Object (File)
Sep 30 2024, 4:12 PM
Unknown Object (File)
Sep 29 2024, 3:21 PM
Unknown Object (File)
Sep 27 2024, 3:49 AM
Subscribers

Details

Summary

parse_notes relies on the caller-supplied callback to initialize "res".
Two callbacks are used in practice, brandnote_cb and note_fctl_cb, and
the latter fails to initialize res. Try to fix it.

Reported by: KMSAN

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 38857
Build 35746: arc lint + arc unit

Event Timeline

I.e. the only effect of this was that the inner loop from check_note potentially did not stopped after fctl note was handled, but continued till the end of the phdrs.

This revision is now accepted and ready to land.Apr 26 2021, 12:03 AM
In D29986#672701, @kib wrote:

I.e. the only effect of this was that the inner loop from check_note potentially did not stopped after fctl note was handled, but continued till the end of the phdrs.

I think so, yes. I will note this in the commit message.