Bug 8939 - MP3 file with invalid id3v2 data causes the scanner to crash
: MP3 file with invalid id3v2 data causes the scanner to crash
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.1
: PC Windows XP
: P5 normal (vote)
: 7.x
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-30 13:14 UTC by Osama Zaidan
Modified: 2009-07-31 10:26 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
File to test (3.31 MB, audio/mpeg)
2008-07-30 13:15 UTC, Osama Zaidan
Details
ScannerLog (7.26 KB, application/octet-stream)
2008-08-01 10:09 UTC, James Richardson
Details
server log (7.63 KB, application/octet-stream)
2008-08-01 10:09 UTC, James Richardson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Osama Zaidan 2008-07-30 13:14:21 UTC
Customer has an MP3 file that causes the scanner to crash. 
File will play in iTunes and Windows media player without any problem.
We checked the file tag using MediaMonkey, but there was nothing unusual about the tag. 
We tried the file on SC 7.0.1 and 7.1 we got same result, the scanner crashes everytime it attempts to scan this file.
Comment 1 Osama Zaidan 2008-07-30 13:15:25 UTC
Created attachment 3710 [details]
File to test
Comment 2 KDF 2008-07-30 13:22:26 UTC
is there a log clip that an be attached?
Comment 3 KDF 2008-07-30 18:44:33 UTC
tag and rename shows no tags for this file.  it seems that MP3::Info is seeking past the end of file while trying harder to find a valid header.
Comment 4 James Richardson 2008-08-01 10:09:01 UTC
Created attachment 3729 [details]
ScannerLog
Comment 5 James Richardson 2008-08-01 10:09:15 UTC
Created attachment 3730 [details]
server log
Comment 6 James Richardson 2008-08-01 10:09:56 UTC
Unable to reproduce a "lockup" with 7.2, however the server did become unresponsive when scanning just this single file.  As soon as the scan was done, the server was back to normal.

Log Attached.
Comment 7 Andy Grundman 2008-08-05 05:17:09 UTC
Can you reproduce with scan.scanner debug?  I don't have any problems with this file, maybe it's a Windows-only issue?

If the scanner actually crashes, what is the crash message?  These logs look fine.
Comment 8 KDF 2008-08-05 09:00:50 UTC
using formats.audio, plus a few warnings in MP3::Info for reference.  After the second case of the warning on line 1603, perl interpreter crashes "encountered a problem and needs to close" dialog box. Warning at 1598 is $off, and at 1603 is $tot.

C:\slim\boom\server>scanner.pl c:\music\bug8939.mp3 --debug formats.audio
[08:57:47.2170] Slim::Formats::MP3::getTag (156) Trying to read ID3v2 tags from C:\music\bug8939.mp3
[08:57:47.2263] Slim::Formats::MP3::getTag (165) Didn't find any ID3v2 tags, trying v1 tags.
[08:57:47.2274] MP3::Info::get_mp3info (1598) Warning: 503316498 at C:\slim\boom\server\lib/MP3/Info.pm line 1598, <$fh> line 1.
[08:57:47.2279] MP3::Info::get_mp3info (1602) Warning: Warning: something's wrong at C:\slim\boom\server\lib/MP3/Info.pm line 1602, <$fh> line 1.
[08:57:47.2283] MP3::Info::get_mp3info (1603) Warning: 503324690 at C:\slim\boom\server\lib/MP3/Info.pm line 1603, <$fh> line 1.
0[08:57:47.2288] MP3::Info::get_mp3info (1624) Warning: Warning: something's wrong at C:\slim\boom\server\lib/MP3/Info.pm line 1624, <$fh> line 1.
[08:57:47.2294] Slim::Formats::MP3::getTag (189) Didn't find audio information in first pass - trying harder.
[08:57:47.2301] MP3::Info::get_mp3info (1598) Warning: 503316498 at C:\slim\boom\server\lib/MP3/Info.pm line 1598, <$fh> line 2.
[08:57:47.2306] MP3::Info::get_mp3info (1602) Warning: Warning: something's wrong at C:\slim\boom\server\lib/MP3/Info.pm line 1602, <$fh> line 2.
[08:57:47.2311] MP3::Info::get_mp3info (1603) Warning: 503365650 at C:\slim\boom\server\lib/MP3/Info.pm line 1603, <$fh> line 2.
Comment 9 Mike Walsh 2008-08-05 11:00:21 UTC
using winamp, i couldn't find any tags at all.  winamp played it fine tho.

i then used lametag to see if it had a lame tag header, and it doesn't.

will SC play a mp3 if it has no lame tag at all?
Comment 10 James Richardson 2008-08-08 09:10:42 UTC
Andy: See KDF's snipet
Comment 11 Andy Grundman 2008-08-08 09:21:38 UTC
OK, it's a Windows-only issue, will try to fix.
Comment 12 Andy Grundman 2008-08-08 13:48:39 UTC
Fixed in change 22482.
Comment 13 Ross Levine 2008-10-14 13:37:41 UTC
Verified, no scanner crash, 7.2.1 - 23530.
Comment 14 James Richardson 2008-12-15 12:34:32 UTC
This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 15 Chris Owens 2009-07-31 10:26:11 UTC
Reduce number of active targets for SC