Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#4832 closed defect (invalid)

close dynlib file handle if erroring out

Reported by: Erik Hovland <erik@…> Owned by: Anduin Withers
Priority: trivial Milestone: 0.21
Component: mythtv Version: 0.21-fixes
Severity: low Keywords:
Cc: Ticket locked: no

Description

When InitFilter? doesn't work and the function FilterManager::LoadFilter?() returns, it should close the open dynlib file handle.

Attachments (1)

libs_libmythtv_filtermanager.cpp-dynlib-file-handle-leak.patch​ (822 bytes) - added by Erik Hovland <erik@…> 16 years ago.
closes dynlib file handle on error return

Download all attachments as: .zip

Change History (4)

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

Attachment: libs_libmythtv_filtermanager.cpp-dynlib-file-handle-leak.patch​ added

closes dynlib file handle on error return

comment:1 Changed 16 years ago by Anduin Withers

Owner: changed from Isaac Richards to Anduin Withers
Status: new → assigned

comment:2 Changed 16 years ago by Anduin Withers

Resolution: → invalid
Status: assigned → closed

Just because it isn’t on the same line as your original patch doesn’t mean it isn’t in there.

comment:3 Changed 16 years ago by Erik Hovland <erik@…>

Whoops. Sorry. I should have looked harder. I just imported my trunk patch tree to my 0.21-fixes tree and started submitted what looked easy. See what I get for being lazy.

Thanks for double checking and sorry again.

Note: See TracTickets for help on using tickets.