mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-25 19:21:06 +00:00
Code Issues 1 Projects Releases Wiki Activity

tag: id3v2: Add more validation to ensure that a char encoding conversion produced a valid UTF-8 string.

Browse source 
Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (parse_insert_string_field),
(parse_split_strings):
Add more validation to ensure that a char encoding conversion
produced a valid UTF-8 string.
This commit is contained in:
Jan Schmidt 2006-02-13 12:00:51 +00:00 committed by Tim-Philipp Müller
parent 05a5fd1e51
commit dc5506d372
1 changed files with 35 additions and 33 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -457,12 +457,27 @@ id3v2_genre_fields_to_taglist (ID3TagsWorking * work, const gchar * tag_name,
return result; return result;
} }
static void
parse_insert_string_field (const gchar * encoding, gchar * data, gint data_size,
GArray * fields)
{
gchar *field;
field = g_convert (data, data_size, "UTF-8", encoding, NULL, NULL, NULL);
if (field && !g_utf8_validate (field, -1, NULL)) {
GST_DEBUG ("%s was bad UTF-8. Ignoring", field);
g_free (field);
field = NULL;
}
if (field)
g_array_append_val (fields, field);
}
static void static void
parse_split_strings (guint8 encoding, gchar * data, gint data_size, parse_split_strings (guint8 encoding, gchar * data, gint data_size,
GArray ** out_fields) GArray ** out_fields)
{ {
GArray *fields = g_array_new (FALSE, TRUE, sizeof (gchar *)); GArray *fields = g_array_new (FALSE, TRUE, sizeof (gchar *));
gchar *field;
gint text_pos; gint text_pos;
gint prev = 0; gint prev = 0;
@ -472,52 +487,46 @@ parse_split_strings (guint8 encoding, gchar * data, gint data_size,
case ID3V2_ENCODING_ISO8859: case ID3V2_ENCODING_ISO8859:
for (text_pos = 0; text_pos < data_size; text_pos++) { for (text_pos = 0; text_pos < data_size; text_pos++) {
if (data[text_pos] == 0) { if (data[text_pos] == 0) {
field = g_convert (data + prev, text_pos - prev + 1, parse_insert_string_field ("ISO-8859-1", data + prev,
"UTF-8", "ISO-8859-1", NULL, NULL, NULL); text_pos - prev + 1, fields);
if (field)
g_array_append_val (fields, field);
prev = text_pos + 1; prev = text_pos + 1;
} }
} }
if (data_size - prev > 0 && data[prev] != 0x00) { if (data_size - prev > 0 && data[prev] != 0x00) {
field = g_convert (data + prev, data_size - prev, parse_insert_string_field ("ISO-8859-1", data + prev,
"UTF-8", "ISO-8859-1", NULL, NULL, NULL); data_size - prev, fields);
if (field)
g_array_append_val (fields, field);
} }
break; break;
case ID3V2_ENCODING_UTF8: case ID3V2_ENCODING_UTF8:
for (prev = 0, text_pos = 0; text_pos < data_size; text_pos++) { for (prev = 0, text_pos = 0; text_pos < data_size; text_pos++) {
if (data[text_pos] == '\0') { if (data[text_pos] == '\0') {
field = g_strndup (data + prev, text_pos - prev + 1); parse_insert_string_field ("UTF-8", data + prev,
if (field) text_pos - prev + 1, fields);
g_array_append_val (fields, field);
prev = text_pos + 1; prev = text_pos + 1;
} }
} }
if (data_size - prev > 0 && data[prev] != 0x00) { if (data_size - prev > 0 && data[prev] != 0x00) {
field = g_strndup (data + prev, data_size - prev); parse_insert_string_field ("UTF-8", data + prev,
if (field) data_size - prev, fields);
g_array_append_val (fields, field);
} }
break; break;
case ID3V2_ENCODING_UTF16: case ID3V2_ENCODING_UTF16:
case ID3V2_ENCODING_UTF16BE: case ID3V2_ENCODING_UTF16BE:
{ {
const gchar *in_encode;
if (encoding == ID3V2_ENCODING_UTF16)
in_encode = "UTF-16";
else
in_encode = "UTF-16BE";
/* Find '\0\0' terminator */ /* Find '\0\0' terminator */
for (text_pos = 0; text_pos < data_size - 1; text_pos += 2) { 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 */ /* found a delimiter */
if (encoding == ID3V2_ENCODING_UTF16) { parse_insert_string_field (in_encode, data + prev,
field = g_convert (data + prev, text_pos - prev + 2, text_pos - prev + 2, fields);
"UTF-8", "UTF-16", NULL, NULL, NULL);
} else {
field = g_convert (data + prev, text_pos - prev + 2,
"UTF-8", "UTF-16BE", NULL, NULL, NULL);
}
if (field)
g_array_append_val (fields, field);
text_pos++; /* Advance to the 2nd NULL terminator */ text_pos++; /* Advance to the 2nd NULL terminator */
prev = text_pos + 1; prev = text_pos + 1;
break; break;
@ -526,15 +535,8 @@ parse_split_strings (guint8 encoding, gchar * data, gint data_size,
if (data_size - prev > 1 && if (data_size - prev > 1 &&
(data[prev] != 0x00 || data[prev + 1] != 0x00)) { (data[prev] != 0x00 || data[prev + 1] != 0x00)) {
/* There were 2 or more non-null chars left, convert those too */ /* There were 2 or more non-null chars left, convert those too */
if (encoding == ID3V2_ENCODING_UTF16) { parse_insert_string_field (in_encode, data + prev,
field = g_convert (data + prev, data_size - prev, data_size - prev, fields);
"UTF-8", "UTF-16", NULL, NULL, NULL);
} else {
field = g_convert (data + prev, data_size - prev,
"UTF-8", "UTF-16BE", NULL, NULL, NULL);
}
if (field)
g_array_append_val (fields, field);
} }
break; break;
} }