Bug 14462 - Music scan terminated unexpectedly (Directory Scan)
: Music scan terminated unexpectedly (Directory Scan)
Status: VERIFIED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.4.0
: PC Ubuntu Linux
: -- normal (vote)
: 7.4.1
Assigned To: Andy Grundman
: Audio::Scan
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-30 15:34 UTC by Darren Warner
Modified: 2009-10-07 11:38 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
wav file causing scanner to fail (159.79 KB, audio/wav)
2009-10-02 05:09 UTC, Ian Dogherty
Details
Crashes 7.4.0 scanner (32.07 KB, application/octet-stream)
2009-10-02 06:47 UTC, Oliver Lineham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Darren Warner 2009-09-30 15:34:52 UTC
Just upgraded to v7.4.0 - r28672, running on Ubuntu 9.04 (x86_64) with perl v5.10.0 and I get a crash when trying to rescan my library.

I have the following in server.log

[09-09-30 15:28:22.8938] Slim::Web::Settings::Server::Basic::handler (79) Initiating scan of type: rescan
[09-09-30 15:28:22.8961] Slim::Music::Import::countImporters (611) Found importer: Slim::Music::PlaylistFolderScan
[09-09-30 15:28:22.8964] Slim::Music::Import::countImporters (611) Found importer: Slim::Music::MusicFolderScan
[09-09-30 15:29:18.0520] Slim::Music::Import::checkScanningStatus (241) scanner is not running, but no progress data available - scanner crashed
[09-09-30 15:29:18.0641] Slim::Schema::wipeCaches (1571) Wiped all in-memory caches.

The last entry in scanner.log is:
[09-09-30 15:29:15.4751] Slim::Utils::Scanner::scanDirectory (340) Adding file:///Media/Music/Library/iTunes/Stevie%20Wonder/Hotter%20Than%20July/Happy%20Birthday.m4a to database.

I've turned on all scan, scan.import, and scan.scanner logging to Debug, but I don't get any additional output.
Comment 1 James Richardson 2009-10-01 08:27:48 UTC
*** Bug 14479 has been marked as a duplicate of this bug. ***
Comment 2 Andy Grundman 2009-10-01 08:58:31 UTC
Maybe something is wrong with that file, can you attach it?
Comment 3 Darren Warner 2009-10-01 23:23:48 UTC
The file plays fine (at least via gstreamer) but faad shows some non-ASCII characters in 'tool' and 'cover' (the m4a was converted from an m4p via FairUse or DeDRMS IIRC). Moving it out the way allows the scanner to complete successfully.

Quite happy to send the file, but just want to make sure you really want me to do this before I turn bugs.slimdevices.com into a music sharing site. Would a direct email/ftp be better?
Comment 4 Ian Dogherty 2009-10-01 23:54:55 UTC
Might be worth saying that the file mine was failing on was a WAV - I moved this WAV and the scanner moved on but failed when it reached another WAV. Again, I'm quite happy to send these on but clearly it's not going to be a DRM issue.

Ta

Ian
Comment 5 Andy Grundman 2009-10-02 05:01:06 UTC
The only way I can fix the bug is if you upload the file(s).  So yes, please. :)
Comment 6 Ian Dogherty 2009-10-02 05:09:15 UTC
Created attachment 5980 [details]
wav file causing scanner to fail

Hi

This is the first WAV file that caused the scanner to fail on my system.

Hope it helps

Ian
Comment 7 SVN Bot 2009-10-02 05:38:08 UTC
 == Auto-comment from SVN commit #442 to the opensource repo by andy ==
 == https://svn.slimdevices.com/opensource?view=revision&revision=442 ==

Bug 14462, fixed reading of WAV files with 18-byte fmt chunks
Comment 8 Oliver Lineham 2009-10-02 06:47:59 UTC
Created attachment 5983 [details]
Crashes 7.4.0 scanner

Also attaching my own funny little WAV that was crashing my 7.4.0 scanner with the same symptoms as above.
Comment 9 Andy Grundman 2009-10-02 08:21:15 UTC
Oliver: your file has the wrong size for the data chunk.  I will make a fix to handle this better.
Comment 10 SVN Bot 2009-10-02 08:33:34 UTC
 == Auto-comment from SVN commit #443 to the opensource repo by andy ==
 == https://svn.slimdevices.com/opensource?view=revision&revision=443 ==

Bug 14462, fixed reading of WAV files with data chunks having the wrong size
Comment 11 Andy Grundman 2009-10-02 08:54:09 UTC
Darren: any chance I can get that m4a file?  If you don't want to attach it here, you can email it to me at andy@slimdevices.com as long as it's under 25MB.
Comment 12 Darren Warner 2009-10-02 21:22:37 UTC
For the record, email with m4a this morning. Let me know if you didn't get it.
Comment 13 SVN Bot 2009-10-03 12:36:40 UTC
 == Auto-comment from SVN commit #444 to the opensource repo by andy ==
 == https://svn.slimdevices.com/opensource?view=revision&revision=444 ==

Fixed bug 14462, fixed crasher when handling duplicate key names of items that are integers
Comment 14 Darren Warner 2009-10-07 11:37:23 UTC
Upgraded to squeezeboxserver_7.4.0_all.deb and the problem went away
Downgraded back to squeezeboxserver_7.4.0_all.deb and the crash recurred

Thank you!!
Comment 15 Darren Warner 2009-10-07 11:38:23 UTC
Oops - first line should have read "Upgraded to squeezeboxserver_7.4.1~28791_all.deb"