mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-01-29 18:48:44 +00:00
Code Issues 1 Projects Releases Wiki Activity

tag: id3v2: Add gst_element_no_more_pads() for proper decodebin behaviour.

Browse source 
Original commit message from CVS:
* gst-libs/gst/tag/gstid3demux.c: (gst_id3demux_add_srcpad):
Add gst_element_no_more_pads() for proper decodebin behaviour.
* gst-libs/gst/tag/id3v2frames.c: (parse_comment_frame),
(parse_text_identification_frame), (parse_split_strings):
Failure to decode some tags is not a GST_ERROR() but a
GST_WARNING()
When iterating over a chunk of text, check that we haven't gone too
far.
This commit is contained in:
Edward Hervey 2006-01-06 11:46:53 +00:00 committed by Tim-Philipp Müller
parent a2415c8edb
commit f34424e933
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
gst-libs/gst/tag/id3v2frames.c
View file

@ -161,12 +161,12 @@ parse_comment_frame (ID3TagsWorking * work)
parse_split_strings (work, encoding, &description, &text);
if (text == NULL || description == NULL) {
GST_ERROR ("Failed to decode comment frame");
GST_WARNING ("Failed to decode comment frame");
goto fail;
}
if (!g_utf8_validate (text, -1, NULL)) {
GST_ERROR ("Converted string is not valid utf-8");
GST_WARNING ("Converted string is not valid utf-8");
goto fail;
} else {
if (strlen (description) > 0 && g_utf8_validate (description, -1, NULL)) {
@ -214,7 +214,7 @@ parse_text_identification_frame (ID3TagsWorking * work)
}
if (text != NULL && !g_utf8_validate (text, -1, NULL)) {
GST_ERROR ("Converted string is not valid utf-8");
GST_WARNING ("Converted string is not valid utf-8");
g_free (text);
text = NULL;
}
@ -332,7 +332,7 @@ parse_split_strings (ID3TagsWorking * work, guint8 encoding,
switch (encoding) {
case ID3V2_ENCODING_ISO8859:
for (text_pos = 4; text_pos < work->parse_size - 1; text_pos++) {
for (text_pos = 4; text_pos < work->parse_size - 5; text_pos++) {
if (work->parse_data[text_pos] == 0) {
*field1 = g_convert ((gchar *) (work->parse_data + 4),
text_pos - 4, "UTF-8", "ISO-8859-1", NULL, NULL, NULL);
@ -356,7 +356,7 @@ parse_split_strings (ID3TagsWorking * work, guint8 encoding,
case ID3V2_ENCODING_UTF16BE:
{
/* Find '\0\0' terminator */
for (text_pos = 4; text_pos < work->parse_size - 2; text_pos++) {
for (text_pos = 4; text_pos < work->parse_size - 6; text_pos++) {
if (work->parse_data[text_pos] == 0 &&
work->parse_data[text_pos + 1] == 0) {
/* found our delimiter */