Opened 12 years ago
Closed 12 years ago
Last modified 12 years ago
#10721 closed Patch - Bug Fix (fixed)
Mythmusic FLAC import fails with large number of files
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | minor | Milestone: | 0.25.1 |
Component: | MythTV - General | Version: | 0.25-fixes |
Severity: | medium | Keywords: | mythmusic flac metadata |
Cc: | Ticket locked: | no |
Description
When scanning for new music (eg when first installing), mythfrontend crashes when importing metadata for a large number of files. Error indicates too many open files. Incorrect data is also populated as metadata cannot be read for a number of files prior to crash.
In metaioflacvorbis.cpp, flacfile is not deleted after use in MetaIOFLACVorbis::read as it is in the other functions that open the file. This results in each file remaining opened after metadata is read.
Patch attached, tested against 0.25-fixes.
Attachments (1)
Change History (9)
Changed 12 years ago by
Attachment: | scanfornewmusiccrash.patch added |
---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Owner: | set to Stuart Morgan <smorgan@…> |
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:4 Changed 12 years ago by
Milestone: | unknown → 0.25.1 |
---|
comment:5 Changed 12 years ago by
I just updated to 0.25.0+fixes.20120525.eb1d4a7 and this does not fix the issue for me.
comment:6 Changed 12 years ago by
Steven you appear to have a different issue as the original reporter indicated their issue was fixed by the patch. Can you open a new ticket, with a description of the symptoms and a log?
this patch does not work for me, I still get the "Could not open file" error