[mythtv-commits] [MythTV/mythtv] 8e3612: MythEDID: fix out of bounds memory access

Scott Theisen noreply at github.com
Sat Jun 25 12:58:27 UTC 2022

  Branch: refs/heads/master
  Home:   https://github.com/MythTV/mythtv
  Commit: 8e36123478cb98f0134bdc4447beb51d8e708385
  Author: Scott Theisen <scott.the.elm at gmail.com>
  Date:   2022-06-25 (Sat, 25 Jun 2022)

  Changed paths:
    M mythtv/libs/libmythui/mythedid.cpp

  Log Message:
  MythEDID: fix out of bounds memory access

found by valgrind while investigating segmentation faults that
occured randomly on startup of mythfrontend.

There are only 4 descriptors in the EDID data structure.
By incorrectly looping 5 times in MythEDID::ParseBaseBlock(),
an offset of 126 is passed to ParseDisplayDescriptor() or
ParseDetailedTimingDescriptor() which will then read past
the end of the 128 byte EDID data structure.

This appears to have always been wrong since it was added in
53d170221983764ae108939cbdb0bcbfc48b6114 .

More information about the mythtv-commits mailing list