Opened 12 years ago

Closed 12 years ago

#5120 closed defect (fixed)

dyn_cast can return 0, check in HandlePAT and UpdateValues

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

dynamic_cast can return 0. The pointer returned should be checked before being dereferenced.

This patch also has a small change for FirewireSignalMonitor::RunTableMonitor?(). That change can be safely ignored.

Attachments (1)

libs_libmythtv_firewiresignalmonitor.cpp-dyncast-can-be-null.patch​ (1.9 KB) - added by Erik Hovland <erik@…> 12 years ago.
check dynamic_cast

Download all attachments as: .zip

Change History (2)

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

Attachment: libs_libmythtv_firewiresignalmonitor.cpp-dyncast-can-be-null.patch​ added

check dynamic_cast

comment:1 Changed 12 years ago by danielk

Resolution: → fixed
Status: new → closed

(In [17201]) Fixes #5120. Adds some null pointer checks.

Note: See TracTickets for help on using tickets.