Opened 13 years ago

Closed 12 years ago

#5135 closed defect (invalid)

ATSCStreamData::DeleteCachedTable might try to use psip after delete

Reported by: Erik Hovland <erik@…> Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

ATSCStreamData::DeleteCachedTable? can delete psip in two seperate conditionals at lines 730 and 741 and then use psip at line 759. It is unlikely. But it is probably a better idea to defer deletion until after psip is used in the look up in the list.

Attachments (1)

libs_libmythtv_atscstreamdata.cpp-avoid-use-after-free.patch (1.7 KB) - added by Erik Hovland <erik@…> 13 years ago.
defer deleting psip until after it is used for the last time

Download all attachments as: .zip

Change History (2)

Changed 13 years ago by Erik Hovland <erik@…>

defer deleting psip until after it is used for the last time

comment:1 Changed 12 years ago by danielk

Resolution: invalid
Status: newclosed

false positive -- the find(psip) method does not dereference the pointer, it's just using it as a key in a lookup table.

Note: See TracTickets for help on using tickets.