OGG Lyric tag crashes player

I believe I have identified a bug present in both the 02.02.28 and 02.02.26 firmware versions regarding long OGG Vorbis comments.  I cannot test if this bug is present in any other previous firmware, as I’ve just recently received this Fuze new.

Some of my OGG files contain a LYRICS comment placed there by a Songbird add-on (MediaMonkey reads the lyrics fine).  When some of these OGG files with embedded lyrics are copied to the Fuze, it will flash white and then restart while refreshing the database.  It continues to restart in a loop until I force it off or connect it to the computer.  Occasionally, it will hang while refreshing instead of restarting (I’ve let it go for 30 minutes while it refreshed).

After removing the LYRICS comment/tag from each OGG file, the Fuze player has no issues.  Is it possible that the longer than average length of the LYRICS tag is causing an overflow, crashing the player?

kccricket wrote:
I believe I have identified a bug present in both the 02.02.28 and 02.02.26 firmware versions regarding long OGG Vorbis comments. 

After removing the LYRICS comment/tag from each OGG file, the Fuze player has no issues.  Is it possible that the longer than average length of the LYRICS tag is causing an overflow, crashing the player?

Yep. Not a ‘bug’ per say, just the way it works. None of the Sansa players like anything in the ‘Comments’ field of a tag. Especially if they’re long, like the entire lyrics of a song.

Media Monkey can use the processing power of your computer to display these. The Fuze’s brain is the size of a house-fly by comparison.

@tapeworm wrote:

Yep. Not a ‘bug’ per say, just the way it works. None of the Sansa players like anything in the ‘Comments’ field of a tag. Especially if they’re long, like the entire lyrics of a song.

Media Monkey can use the processing power of your computer to display these. The Fuze’s brain is the size of a house-fly by comparison.

I feel differently about the issue and disagree with you.  I understand that embedded programming is much different than the kind of scripted programming I’m used to.  However, I would still consider this a bug.  Again, I’m not an embedded systems programmer, but this behavior seems to indicate a lack of bounds checking somewhere in the comment parser, which is always a bug in my opinion.

I believe I will simply submit this to Sandisk support.

kccricket wrote:

@tapeworm wrote:

Yep. Not a ‘bug’ per say, just the way it works. None of the Sansa players like anything in the ‘Comments’ field of a tag. Especially if they’re long, like the entire lyrics of a song.

Media Monkey can use the processing power of your computer to display these. The Fuze’s brain is the size of a house-fly by comparison.


I feel differently about the issue and disagree with you.  I understand that embedded programming is much different than the kind of scripted programming I’m used to.  However, I would still consider this a bug.  Again, I’m not an embedded systems programmer, but this behavior seems to indicate a lack of bounds checking somewhere in the comment parser, which is always a bug in my opinion.

 

I believe I will simply submit this to Sandisk support.

And they will probably tell you that you can’t expect lyric support in a “value player”, which is what they describe the Sansa line as.

@marvin_martian wrote:


@kccricket wrote:

@tapeworm wrote:

Yep. Not a ‘bug’ per say, just the way it works. None of the Sansa players like anything in the ‘Comments’ field of a tag. Especially if they’re long, like the entire lyrics of a song.

Media Monkey can use the processing power of your computer to display these. The Fuze’s brain is the size of a house-fly by comparison.


I feel differently about the issue and disagree with you.  I understand that embedded programming is much different than the kind of scripted programming I’m used to.  However, I would still consider this a bug.  Again, I’m not an embedded systems programmer, but this behavior seems to indicate a lack of bounds checking somewhere in the comment parser, which is always a bug in my opinion.

 

I believe I will simply submit this to Sandisk support.


And they will probably tell you that you can’t expect lyric support in a “value player”, which is what they describe the Sansa line as.

The best retort for this answer would be “Hmmmm…I wonder if my lawyer will feel the same way?”

@marvin_martian wrote:

And they will probably tell you that you can’t expect lyric support in a “value player”, which is what they describe the Sansa line as.

I think, perhaps, you are missing my point.  I’m not expecting lyric support in a “value player.”  I don’t care if the lyrics show up on the Sansa.  However, I do not feel it’s appropriate for the player to choke on very long comments.  I feel it should simply ignore them.

OGG support is a recent development in this line of players.  As with any new software, the developer’s implementation may be slightly unstable or have a few bugs (e.g. the recent ReplayGain bugs).  I think it’s great that Sansa finally added OGG and FLAC support to their more popular players.  It’s a smart move considering the popularity of custom firmware for their devices.

I’m certainly not upset that the device may have this bug.  Software development is a gradual process, and bugs can’t be fixed unless people find and report them.  I feel that posts along the lines of “I tried OGGs with lyrics, and they worked fine” or “I just tried this and, it breaks mine, too” are more helpful than the sort of (admittedly brief) discussion appearing in this thread so far.

kccricket wrote:


I think, perhaps, you are missing my point.  I’m not expecting lyric support in a “value player.”  I don’t care if the lyrics show up on the Sansa.  However, I do not feel it’s appropriate for the player to choke on very long comments.  I feel it should simply ignore them.

 

That’s an entirely fair point. My comment was more a barb aimed at Sansa than anything else, the “value player” comment a reference to their statement that they will not work further on their players that play off-pitch because they are “value players” . I hope you didn’t misinterpret my rolleyes as being aimed at you.


kccricket wrote:


 

I’m certainly not upset that the device may have this bug.  Software development is a gradual process, and bugs can’t be fixed unless people find and report them.  I feel that posts along the lines of “I tried OGGs with lyrics, and they worked fine” or “I just tried this and, it breaks mine, too” are more helpful than the sort of (admittedly brief) discussion appearing in this thread so far.

Again, a fair point.:wink:

@marvin_martian wrote:

That’s an entirely fair point. My comment was more a barb aimed at Sansa than anything else, the “value player” comment a reference to their statement that they will not work further on their players that play off-pitch because they are “value players” . I hope you didn’t misinterpret my rolleyes as being aimed at you.

Ah, yea, I misinterpreted the whole comment.  I’m new to the forums, so I didn’t get the “value player” reference. :dizzy_face:
Thanks for clarifying.

kccricket wrote:


@marvin_martian wrote:

That’s an entirely fair point. My comment was more a barb aimed at Sansa than anything else, the “value player” comment a reference to their statement that they will not work further on their players that play off-pitch because they are “value players” . I hope you didn’t misinterpret my rolleyes as being aimed at you.


Ah, yea, I misinterpreted the whole comment.  I’m new to the forums, so I didn’t get the “value player” reference. :dizzy_face:
Thanks for clarifying.

No problem! :smiley: