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

	Unknown Object (File)
	Mon, Jul 27, 10:10 PM

	Unknown Object (File)
	Sun, Jul 26, 2:27 AM

	Unknown Object (File)
	Thu, Jul 23, 2:24 AM

	Unknown Object (File)
	Wed, Jul 22, 10:44 PM

	Unknown Object (File)
	Tue, Jul 21, 10:41 AM

	Unknown Object (File)
	Sun, Jul 19, 12:03 PM

	Unknown Object (File)
	Sat, Jul 18, 10:07 AM

	Unknown Object (File)
	Jul 7 2026, 9:38 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...