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

None

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

Lint

Lint Skipped

Unit

Tests Skipped

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	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 subscribed.Feb 1 2019, 6:44 PM

kib added inline comments.

contrib/elftoolchain/readelf/readelf.c
2736	This is weird, why not do &= ~ ?
2741	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. · 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	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

contrib/

elftoolchain/

readelf/

196 line(s)

Diff 53509

contrib/elftoolchain/readelf/readelf.c

Loading...