Bug 165528 - bogus module_verify_elf check rejects valid .ko files
Summary: bogus module_verify_elf check rejects valid .ko files
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Howells
QA Contact: Brian Brock
URL:
Whiteboard:
: 166828 (view as bug list)
Depends On:
Blocks: 165416
TreeView+ depends on / blocked
 
Reported: 2005-08-10 07:55 UTC by Roland McGrath
Modified: 2007-11-30 22:11 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2005-08-15 21:13:59 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
fix for bogus elf layout assertion in module verification (538 bytes, patch)
2005-08-10 07:55 UTC, Roland McGrath
no flags Details | Diff

Description Roland McGrath 2005-08-10 07:55:25 UTC
Description of problem:

A bug in one of the sanity checks on kernel module files done by the module
signature checking stuff causes it to reject valid .ko files that happen to have
their section headers at the end of the file.  It's pure luck that older tools
haven't produced files with this layout before, and it could bite on future
builds after binutils/elfutils updates.  This bug should be cloned to all the FC
and RHEL release streams that have the module signing code.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Fix attached.

Comment 1 Roland McGrath 2005-08-10 07:55:37 UTC
Created attachment 117597 [details]
fix for bogus elf layout assertion in module verification

Comment 2 Frank Ch. Eigler 2005-08-11 17:08:21 UTC
Would a workaround in elfutils (or whoever is creating these newfangled files)
be prudent in the mean time, to avoid triggering the bug?

Comment 3 Roland McGrath 2005-08-15 21:13:59 UTC
Dave put the fix in.

Comment 4 Dave Jones 2005-08-31 23:12:27 UTC
*** Bug 166828 has been marked as a duplicate of this bug. ***


Note You need to log in before you can comment on or make changes to this bug.