Modify ↓

Ticket #5502 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

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

libs_libmythui-check-dyn_cast.patch​ (2.7 KB) - added by Erik Hovland <erik@…> 4 years ago.
check dynamic_cast in libmythui
libs_libmythui-check-dyn_cast-ticket-5502.patch​ (2.2 KB) - added by Erik Hovland <erik@…> 4 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@…> 4 years ago.
Some cruft crepted into the patch. This version should be cruft free!

Change History

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

check dynamic_cast in libmythui

comment:1 Changed 4 years ago by stuartm

  • Owner changed from ijr to stuartm
  • Status changed from new to assigned

comment:2 Changed 4 years ago by stuartm

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

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

Latest version of the dyn cast patch

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

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

comment:3 Changed 4 years ago by stuartm

  • Status changed from assigned to closed
  • Resolution set to fixed

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

View ↑

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.