Ticket #4958 (closed defect: invalid)
Opened 4 years ago
Last modified 4 years ago
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
Change History
Changed 4 years ago by Erik Hovland <erik@…>
- Attachment libs_libmythdvdnav_dvd_input.c-close-dl-file-handle.patch added
comment:1 Changed 4 years ago by danielk
- Owner changed from ijr to danielk
- Status changed from new to assigned
comment:2 Changed 4 years ago by danielk
- Status changed from assigned to infoneeded
Erik, I can find no documentation that supports the idea that this will work. Do you have a link ?
comment:3 Changed 4 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 4 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 4 years ago by nigel
- Status changed from infoneeded to closed
- Resolution set to invalid
calls a dlclose() at the end of the function so the handle ref count is decremented.