diff options
| author | Jan Schmidt <thaytan@mad.scientist.com> | 2006-02-04 13:30:12 +0000 |
|---|---|---|
| committer | Jan Schmidt <thaytan@mad.scientist.com> | 2006-02-04 13:30:12 +0000 |
| commit | 4836bfbcf703e9ee38432a43a83a1c901b7eb404 (patch) | |
| tree | 0125104d274ff5f40c37ea7022ed2334d2cb5269 /gst/id3demux | |
| parent | 9410acd1fad2c01cdb0291a82345c7b58dc74ae2 (diff) | |
gst/id3demux/id3v2frames.c: Adjust for data length indicators when parsing (Fixes #329810)
Original commit message from CVS:
* gst/id3demux/id3v2frames.c: (id3demux_id3v2_parse_frame),
(parse_split_strings):
Adjust for data length indicators when parsing (Fixes #329810)
Fix stupid bug parsing UTF-8 tag text.
Output tag strings with multiple fields as multiple tags, so the
app gets all the data.
Diffstat (limited to 'gst/id3demux')
| -rw-r--r-- | gst/id3demux/id3v2frames.c | 23 |
1 files changed, 14 insertions, 9 deletions
diff --git a/gst/id3demux/id3v2frames.c b/gst/id3demux/id3v2frames.c index 3ebfc07d..48fe9f1c 100644 --- a/gst/id3demux/id3v2frames.c +++ b/gst/id3demux/id3v2frames.c @@ -100,8 +100,9 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work) frame_data_size); return FALSE; } - } else - work->parse_size = frame_data_size; + } + + work->parse_size = frame_data_size; if (work->frame_flags & ID3V2_FRAME_FORMAT_COMPRESSION) { #ifdef HAVE_ZLIB @@ -130,7 +131,7 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work) return FALSE; #endif } else { - work->parse_data = work->hdr.frame_data; + work->parse_data = frame_data; } if (work->frame_id[0] == 'T') { @@ -151,9 +152,8 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work) /* Unique file identifier */ } - if (work->frame_flags & ID3V2_FRAME_FORMAT_COMPRESSION) { + if (work->frame_flags & ID3V2_FRAME_FORMAT_COMPRESSION) g_free (work->parse_data); - } if (tag_str != NULL) { /* g_print ("Tag %s value %s\n", tag_name, tag_str); */ @@ -165,8 +165,13 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work) /* Genre strings need special treatment */ result |= id3v2_genre_fields_to_taglist (work, tag_name, tag_fields); } else { - tag_str = g_array_index (tag_fields, gchar *, 0); - result |= id3v2_tag_to_taglist (work, tag_name, tag_str); + gint t; + + for (t = 0; t < tag_fields->len; t++) { + tag_str = g_array_index (tag_fields, gchar *, t); + if (tag_str != NULL && tag_str[0] != '\0') + result |= id3v2_tag_to_taglist (work, tag_name, tag_str); + } } free_tag_strings (tag_fields); } @@ -484,7 +489,7 @@ parse_split_strings (guint8 encoding, gchar * data, gint data_size, break; case ID3V2_ENCODING_UTF8: for (prev = 0, text_pos = 0; text_pos < data_size; text_pos++) { - if (data[text_pos]) { + if (data[text_pos] == '\0') { field = g_strndup (data + prev, text_pos - prev + 1); if (field) g_array_append_val (fields, field); @@ -502,7 +507,7 @@ parse_split_strings (guint8 encoding, gchar * data, gint data_size, { /* Find '\0\0' terminator */ for (text_pos = 0; text_pos < data_size - 1; text_pos += 2) { - if (data[text_pos] == 0 && data[text_pos + 1] == 0) { + if (data[text_pos] == '\0' && data[text_pos + 1] == '\0') { /* found a delimiter */ if (encoding == ID3V2_ENCODING_UTF16) { field = g_convert (data + prev, text_pos - prev + 2, |