Differential D19053

readelf: use table-based DT_FLAGS and DT_FLAGS_1 decoding
ClosedPublic
Actions

Authored by emaste on Feb 1 2019, 3:09 PM.

Tags

None

Referenced Files

	F173061894: D19053.id.diff
	Wed, Sep 23, 7:00 AM

	Unknown Object (File)
	Tue, Sep 22, 8:03 PM

	Unknown Object (File)
	Mon, Sep 21, 9:23 PM

	Unknown Object (File)
	Mon, Sep 21, 9:19 PM

	Unknown Object (File)
	Mon, Sep 21, 9:17 PM

	Unknown Object (File)
	Thu, Sep 10, 12:00 PM

	Unknown Object (File)
	Sat, Sep 5, 8:35 AM

	Unknown Object (File)
	Sat, Sep 5, 2:14 AM

Subscribers

Details

Reviewers

markj
kaiw
jkoshy_users.sourceforge.net
brooks

Commits

rS343665: readelf: use table-based DT_FLAGS and DT_FLAGS_1 decoding

Summary

Fewer lines of code and more maintainable.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste created this revision.Feb 1 2019, 3:09 PM

emaste added a child revision: D19054: readelf: decode FreeBSD note types.Feb 1 2019, 3:13 PM

LGTM

contrib/elftoolchain/readelf/readelf.c
2741 ↗	(On Diff #53509)	Would this make more sense to the user as something like: printf(" unknown(0x%jx)", (uintmax_t)val);

This revision is now accepted and ready to land.Feb 1 2019, 6:18 PM

kib added a subscriber: kib.Feb 1 2019, 6:44 PM

kib added inline comments.

contrib/elftoolchain/readelf/readelf.c
2736 ↗	(On Diff #53509)	This is weird, why not do &= ~ ?
2741 ↗	(On Diff #53509)	It would make more sense to print each unknown flag individually. At least I do find it more useful, from my experience with the Intel CPUID bits and DMAR capabilities reports.

Closed by commit rS343665: readelf: use table-based DT_FLAGS and DT_FLAGS_1 decoding (authored by emaste). · Explain WhyFeb 1 2019, 8:28 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptFeb 1 2019, 8:28 PM

emaste added inline comments.Feb 1 2019, 8:30 PM

contrib/elftoolchain/readelf/readelf.c
2741 ↗	(On Diff #53509)	Not a bad idea, I will keep it in mind for a followup

dim mentioned this in rS367466: Sync up elftoolchain with head, except for the capsicum-related commits,.Nov 7 2020, 11:58 PM

Revision Contents
Changeset List

Path

Size

head/

contrib/

elftoolchain/

readelf/

196 lines

Diff 53524

head/contrib/elftoolchain/readelf/readelf.c

Loading...