Page MenuHomeFreeBSD
Differential D2338

readelf: avoid division by zero for files with invalid sh_entsize
ClosedPublic
Actions

Authored by emaste on Apr 20 2015, 8:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 12:01 AM
Unknown Object (File)
Sun, Dec 1, 7:34 AM
Unknown Object (File)
Thu, Nov 28, 8:49 PM
Unknown Object (File)
Thu, Nov 28, 8:15 PM
Unknown Object (File)
Fri, Nov 22, 4:15 AM
Unknown Object (File)
Wed, Nov 13, 11:19 AM
Unknown Object (File)
Nov 8 2024, 10:05 AM
Unknown Object (File)
Nov 5 2024, 10:17 PM
View All Files
Subscribers
brooks
freebsd-toolchain-list

Details

Reviewers
brooks
Commits
rS285845: readelf: avoid division by zero on section entry size
Summary

Variations reported in:
https://sourceforge.net/p/elftoolchain/tickets/439
https://sourceforge.net/p/elftoolchain/tickets/444
https://sourceforge.net/p/elftoolchain/tickets/445
https://sourceforge.net/p/elftoolchain/tickets/467

Diff Detail

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

Event Timeline

emaste updated this revision to Diff 4930.Apr 20 2015, 8:43 PM
emaste retitled this revision from to readelf: avoid division by zero for files with invalid sh_entsize.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added a subscriber: Unknown Object (MLST).
brooks accepted this revision.Apr 20 2015, 8:50 PM
brooks added a reviewer: brooks.
brooks added a subscriber: brooks.

Looks good to me.

This revision is now accepted and ready to land.Apr 20 2015, 8:50 PM
emaste updated this revision to Diff 7069.Jul 19 2015, 11:13 AM
emaste edited edge metadata.

Add a get_ent_count helper to check for 0 entsize instead of expanding the check inline everywhere.

This revision now requires review to proceed.Jul 19 2015, 11:13 AM
brooks added a comment.Jul 23 2015, 5:32 PM

Is this still live?

emaste added a comment.Jul 23 2015, 5:47 PM

Yes - sorry for the delay. I realized I had a newer implementation that factored the divide-by-zero checks into a helper function, and uploaded the new diff a few days ago.

brooks accepted this revision.Jul 23 2015, 7:32 PM
brooks edited edge metadata.

Looks good to me.

This revision is now accepted and ready to land.Jul 23 2015, 7:32 PM
Closed by commit rS285845: readelf: avoid division by zero on section entry size (authored by emaste). · Explain WhyJul 24 2015, 6:01 PM
This revision was automatically updated to reflect the committed changes.
emaste added a comment.Jul 27 2015, 6:57 PM

Upstream commit here: https://sourceforge.net/p/elftoolchain/code/3227/

Revision Contents
Changeset List

PathSize
contrib/
elftoolchain/
readelf/
62 lines
head/
contrib/
elftoolchain/
readelf/
62 lines

Diff 7269

View Options

contrib/elftoolchain/readelf/readelf.c

Loading...
View Options

head/contrib/elftoolchain/readelf/readelf.c

Loading...