Bug 9634 - The attached MPC file causes Controler to fail (character 0x00 in artist name)
: The attached MPC file causes Controler to fail (character 0x00 in artist name)
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.4.0
: PC Windows XP
: P1 major (vote)
: 7.4.0
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-10-02 12:05 UTC by Bruno Liechti
Modified: 2009-10-05 14:32 UTC (History)
6 users (show)

See Also:
Category: ---


Attachments
shows appearence of SqueezeCenter with MPC file using ";" as separator (112.56 KB, image/png)
2008-10-02 14:27 UTC, Bruno Liechti
Details
sample track MPC using ";" as separator (3.05 MB, application/octet-stream)
2008-10-02 14:32 UTC, Bruno Liechti
Details
one entry for two artists (13.38 KB, image/png)
2008-10-31 04:44 UTC, Bruno Liechti
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Bruno Liechti 2008-10-02 12:05:39 UTC
MPC files tagged as "Czech Philharmonic Orchestra; Charles Mackerras" will produce one single entry in the library which looks like "Czech Philharmonic Orchestra? Charles Mackerras". When I select this entry on the Squeezebox Controller it crashes and I have to reboot. SqueezeCenter doesn't crash and plays the album correctly but the display is equal to the controller.
Comment 1 James Richardson 2008-10-02 12:24:43 UTC
Please provide us a sample track for evaluation.

What version of SC are you running?
What firmware version is on your Controller?
Comment 2 Bruno Liechti 2008-10-02 14:06:24 UTC
(1) I'm using SqueezeCenter 7.3 - 23377 (it's the same with older versions)

(2) Firmware version of Duet Receiver is 48

(3) Controller runs version 7.2 r2920
Comment 3 Bruno Liechti 2008-10-02 14:27:02 UTC
Created attachment 4094 [details]
shows appearence of SqueezeCenter with MPC file using ";" as separator
Comment 4 Bruno Liechti 2008-10-02 14:32:57 UTC
Created attachment 4095 [details]
sample track MPC using ";" as separator
Comment 5 James Richardson 2008-10-02 15:08:32 UTC
Verified, when the attached file is the ONLY file in the library, as soon as I click PLAY via the controller, it will fall off the network or act badly.

Playing the track from the WEB UI, I get a repeating 4 second clip.

Verified on 7.3 & 7.2.1
Comment 6 James Richardson 2008-10-02 15:41:23 UTC
the same file plays fine outside of SC.  

SB 2/3, BOOM, TP do not lock up when playing this track.  However the SC still does not properly transcode the MPC file all the time.

Tested:
AIFF / mppdec = FAIL
FLAC / mppdec/flac = PLAY
WAV / mppdec = FAIL
Comment 7 Michael Herger 2008-10-02 22:13:10 UTC
Please don't preset targets. Thanks.
Comment 8 James Richardson 2008-10-06 06:34:21 UTC
Changing summary, was 'multiple items in tags doesn't work with MPC files'
Comment 9 James Richardson 2008-10-30 14:16:23 UTC
Please retest with the latest nightly build, there were a bunch of transcoding fixes checked into the 7.3 branch
Comment 10 Bruno Liechti 2008-10-31 04:39:59 UTC
retestetd with: SqueezeCenter 7.3 - 23752, Controller 7.3 r3220, Receiver 52

-transcodes properly from beginning to the end
-controller no longer crashes when selecting MPC files
-the only remaining issue is display behavior. Two artists tagged with ";" as a separator still do not show up as two different library entries in Squeezecenter and Controller (have a look at the attachment)
Comment 11 Bruno Liechti 2008-10-31 04:44:52 UTC
Created attachment 4189 [details]
one entry for two artists
Comment 12 James Richardson 2008-10-31 13:34:34 UTC
controller r3250 will cause the controller to lose sync with the receiver, may or may not recover.
Comment 13 Bruno Liechti 2008-11-01 10:00:03 UTC
My former post was wrong. I second that the controller is still losing sync (reboot necessary). 

The files are played correctly though. It is just not possible to access them on the controller through the menu level which contains the ";" (artists).
Comment 14 Ben Klaas 2008-11-02 12:20:16 UTC
this isn't going to be fixed by 7.3. retarget for 7.3.1
Comment 15 Michael Herger 2008-11-18 08:04:36 UTC
Bruno - the separator in that file isn't a semicolon, but a character 0x00. Replacing it with a semicolon correctly splits the two artists.
Comment 16 Ben Klaas 2009-01-05 10:15:22 UTC
with only these exceptions, my bug list is being retargetted for 8.0

for 7.3.2 (very soon):
https://bugs-archive.lyrion.org/show_bug.cgi?id=7186 Browse albums/artists/genres/  years should remember previous state 

for 7.3.3 (later, but before 8.0):
https://bugs-archive.lyrion.org/show_bug.cgi?id=7139 Add editable "Network Settings" page

https://bugs-archive.lyrion.org/show_bug.cgi?id=7502 Add setup mode to set Static IPs (even if DHCP is present)

https://bugs-archive.lyrion.org/show_bug.cgi?id=7829 Jive menu broken if BLOCK_SIZE == list count + 1 and "Pla...

https://bugs-archive.lyrion.org/show_bug.cgi?id=8878 Party Mode

https://bugs-archive.lyrion.org/show_bug.cgi?id=10064 SqueezeCenter doesn't always auto-generate playerstatus w... (which will fix 8300)

https://bugs-archive.lyrion.org/show_bug.cgi?id=6334 add applet for network diagnostics (currently assigned to Tom)
Comment 17 Andy Grundman 2009-07-30 21:58:18 UTC
This bug should be fixed by Audio::Scan which handles APEv2 tags with null separators.
Comment 18 James Richardson 2009-10-05 14:32:28 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.