Opened 11 years ago

Closed 11 years ago

#5705 closed defect (fixed)

mythweather-defects

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

Description

There is a minor memory leak in WeatherSource::probeTimeouts().

There are also a few classes that do not initialize all of their member variables in their ctors

Attachments (2)

mythweather-defects.patch (2.1 KB) - added by Erik Hovland <erik@…> 11 years ago.
closes mem leak by using stack allocated var instead of heap allocated memory
mythweather-uninit.patch (5.7 KB) - added by Erik Hovland <erik@…> 11 years ago.
initialize member vars in ctors

Download all attachments as: .zip

Change History (4)

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

Attachment: mythweather-defects.patch added

closes mem leak by using stack allocated var instead of heap allocated memory

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

Attachment: mythweather-uninit.patch added

initialize member vars in ctors

comment:1 Changed 11 years ago by stuartm

(In [18295]) Fixes a potential memory leak and general ugliness in WeatherSource::probeTimeouts. Refs #5705

comment:2 Changed 11 years ago by stuartm

Resolution: fixed
Status: newclosed

(In [18394]) Fix missing initialisation of member variables in mythweather. Patch from Erik Hovland. Closes #5705

Note: See TracTickets for help on using tickets.