Page MenuHomeFreeBSD
Differential D20702

libelftc: Use size_t for string table size hints.
ClosedPublic
Actions

Authored by markj on Jun 19 2019, 8:54 PM.
Tags
None
Referenced Files
F156751462: D20702.id58807.diff
Sat, May 16, 2:49 AM
Unknown Object (File)
Thu, May 14, 2:54 AM
Unknown Object (File)
Thu, May 7, 9:42 PM
Unknown Object (File)
Thu, May 7, 12:04 PM
Unknown Object (File)
Sun, May 3, 2:26 PM
Unknown Object (File)
Sat, May 2, 5:36 PM
Unknown Object (File)
Mon, Apr 27, 4:46 PM
Unknown Object (File)
Mon, Apr 27, 3:37 PM
View All Files
Subscribers
imp

Details

Reviewers
emaste
Group Reviewers
manpages
Commits
rS349421: libelftc: Consistently use size_t for string table offsets and sizes.
Summary

String table offsets are returned as a size_t, so we might as well be
consistent.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Jun 19 2019, 8:54 PM
Harbormaster completed remote builds in B24943: Diff 58807.Jun 19 2019, 8:54 PM
markj added a reviewer: emaste.Jun 19 2019, 8:54 PM
markj planned changes to this revision.Jun 19 2019, 8:58 PM

This is incomplete.

markj updated this revision to Diff 58808.Jun 19 2019, 9:15 PM

Consistently use (s)size_t for table size and offsets.

Deleted entries have a negative offset, so we use ssize_t there.

Harbormaster completed remote builds in B24944: Diff 58808.Jun 19 2019, 9:15 PM
emaste accepted this revision.Jun 21 2019, 1:13 PM
This revision is now accepted and ready to land.Jun 21 2019, 1:13 PM
Closed by commit rS349421: libelftc: Consistently use size_t for string table offsets and sizes. (authored by markj). · Explain WhyJun 26 2019, 4:32 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rS349421: libelftc: Consistently use size_t for string table offsets and sizes..
Herald added a reviewer: manpages. · View Herald TranscriptJun 26 2019, 4:32 PM
Herald added a subscriber: imp. · View Herald Transcript
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

PathSize
head/
contrib/
elftoolchain/
libelftc/
35 lines
6 lines
4 lines

Diff 59062

View Options

head/contrib/elftoolchain/libelftc/elftc_string_table.c

Loading...
View Options

head/contrib/elftoolchain/libelftc/elftc_string_table_create.3

Loading...
View Options

head/contrib/elftoolchain/libelftc/libelftc.h

Loading...