Opened 18 years ago

Closed 18 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@…> 18 years ago.
defer deleting psip until after it is used for the last time

Download all attachments as: .zip

Change History (2)

by Erik Hovland <erik@…>, 18 years ago

Attachment: libs_libmythtv_atscstreamdata.cpp-avoid-use-after-free.patch​ added

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

comment:1 by danielk, 18 years ago

Resolution: → invalid
Status: new → closed

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.