Opened 12 years ago

Closed 12 years ago

#4958 closed defect (invalid)

dynlib dvdcss_library handle leak

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

Description

The dynlib handle to the dvdcss library will be leaked if dlclose is not called on the library when finished. The dlclose will cause the ref count on the handle to be decremented so that the system can reclaim it when it needs to.

I swear this is the last time I will submit a ticket for a dlclose defect in libmythdvdnav.

Note that this is a very minor leak as it will not cause the leak to keep growing. It is only to be as correct as possible.

Attachments (1)

libs_libmythdvdnav_dvd_input.c-close-dl-file-handle.patch (996 bytes) - added by Erik Hovland <erik@…> 12 years ago.
calls a dlclose() at the end of the function so the handle ref count is decremented.

Download all attachments as: .zip

Change History (6)

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

calls a dlclose() at the end of the function so the handle ref count is decremented.

comment:1 Changed 12 years ago by danielk

Owner: changed from Isaac Richards to danielk
Status: newassigned

comment:2 Changed 12 years ago by danielk

Status: assignedinfoneeded

Erik, I can find no documentation that supports the idea that this will work. Do you have a link ?

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

Hrm, well I was involved in a discussion on the mplayer libdvdnav list that concluded in saying it wouldn't work. So I am inclined to drop the patch and mark this as invalid.

The discussion thread: http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2008-April/000425.html

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

Please close this ticket. It is unlikely that this is a leak that can be closed (or that it is a leak at all).

comment:5 Changed 12 years ago by Nigel

Resolution: invalid
Status: infoneededclosed
Note: See TracTickets for help on using tickets.