tag: id3v2: A track/volume number or count of 0 does not make sense, just ignore it along with negati...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (id3v2_tag_to_taglist):
A track/volume number or count of 0 does not make sense,
just ignore it along with negative numbers (a tag might
only contain a track count without a track number).
This commit is contained in:
Tim-Philipp Müller 2006-05-28 10:05:47 +00:00 committed by Tim-Philipp Müller
parent 7e7ee7278b
commit 6f5e16f4ba

View file

@ -453,8 +453,8 @@ id3v2_tag_to_taglist (ID3TagsWorking * work, const gchar * tag_name,
gint current, total; gint current, total;
if (sscanf (tag_str, "%d/%d", &current, &total) == 2) { if (sscanf (tag_str, "%d/%d", &current, &total) == 2) {
if (total < 0) { if (total <= 0) {
GST_WARNING ("Ignoring negative value for total %d in tag %s", GST_WARNING ("Ignoring invalid value for total %d in tag %s",
total, tag_name); total, tag_name);
} else { } else {
if (strcmp (tag_name, GST_TAG_TRACK_NUMBER) == 0) { if (strcmp (tag_name, GST_TAG_TRACK_NUMBER) == 0) {
@ -472,9 +472,9 @@ id3v2_tag_to_taglist (ID3TagsWorking * work, const gchar * tag_name,
break; break;
} }
if (current < 0) if (current <= 0) {
GST_WARNING ("Ignoring negative value %d in tag %s", current, tag_name); GST_WARNING ("Ignoring invalid value %d in tag %s", current, tag_name);
else { } else {
gst_tag_list_add (tag_list, GST_TAG_MERGE_APPEND, tag_name, current, gst_tag_list_add (tag_list, GST_TAG_MERGE_APPEND, tag_name, current,
NULL); NULL);
} }