Opened 18 years ago

Closed 18 years ago

#5502 closed defect (fixed)

check dynamic_cast in libmythui

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

Description

libmythui has a few places where the dynamic_cast is not checked to see if it returned null.

The attached patch collects those instances and does the check.

Attachments (3)

libs_libmythui-check-dyn_cast.patch​ (2.7 KB ) - added by Erik Hovland <erik@…> 18 years ago.
check dynamic_cast in libmythui
libs_libmythui-check-dyn_cast-ticket-5502.patch​ (2.2 KB ) - added by Erik Hovland <erik@…> 18 years ago.
Latest version of the dyn cast patch
libs_libmythui-check-dyn_cast-ticket-5502.2.patch​ (1.5 KB ) - added by Erik Hovland <erik@…> 18 years ago.
Some cruft crepted into the patch. This version should be cruft free!

Download all attachments as: .zip

Change History (6)

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

Attachment: libs_libmythui-check-dyn_cast.patch​ added

check dynamic_cast in libmythui

comment:1 by stuartm, 18 years ago

Owner: changed from Isaac Richards to stuartm
Status: new → assigned

comment:2 by stuartm, 18 years ago

(In [17783]) Check the dynamic cast of buttontemplate, thanks to Eric Hovland for the heads up, Refs #5502

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

Attachment: libs_libmythui-check-dyn_cast-ticket-5502.patch​ added

Latest version of the dyn cast patch

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

Attachment: libs_libmythui-check-dyn_cast-ticket-5502.2.patch​ added

Some cruft crepted into the patch. This version should be cruft free!

comment:3 by stuartm, 18 years ago

Resolution: → fixed
Status: assigned → closed

(In [18397]) Check pointers following dynamic cast in MythUIButton. Patch by Erik Hovland. Closes #5502

Note: See TracTickets for help on using tickets.