fwts: acpidump can overrun on MADT if the table contains garbage at end and length is incorrect

Bug #1278422 reported by Colin Ian King
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Firmware Test Suite
Fix Released
High
Colin Ian King

Bug Description

With some firmware where MADT states it is longer than it really is then fwts acpidump can overrun and break because of garbage at the end of the table. We need to check for invalid sub table types and invalid sub table lengths and break out of the loop when we hit these garbage values.

Changed in fwts:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Colin King (colin-king)
Revision history for this message
Colin Ian King (colin-king) wrote :
Changed in fwts:
milestone: none → 14.02.00
Revision history for this message
Colin Ian King (colin-king) wrote :

Fix committed, commit 5a8ce18059de2d1c5cdd25437c572949d6536b48

Changed in fwts:
status: In Progress → Fix Committed
Alex Hung (alexhung)
Changed in fwts:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.