luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-01-27 06:26:15 +01:00
Code Issues Packages Projects Releases Wiki Activity

g_settings_get: check validity of format string

Browse Source 
Allow only format strings that copy all values (i.e, don't contain '&'),
as the returned pointers might become invalid in some rare cases.

Since this is technically an API break, this patch only prints a
critical when a faulty format string is detected, but still fetches the
values.

https://bugzilla.gnome.org/show_bug.cgi?id=719979
This commit is contained in:
Lars Uebernickel 2013-12-06 16:01:48 +01:00
parent c07eccd9c2
commit 396d40af23
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gio/gsettings.c
View File

@ -1528,6 +1528,13 @@ g_settings_get (GSettings *settings,
value = g_settings_get_value (settings, key); value = g_settings_get_value (settings, key);
if (!g_variant_check_format_string (value, format, TRUE))
{
g_critical ("%s: invalid format string for key '%s' from schema '%s'. "
"This call will probably stop working with a future version of glib.",
G_STRFUNC, key, g_settings_schema_get_id (settings->priv->schema));
}
va_start (ap, format); va_start (ap, format);
g_variant_get_va (value, format, NULL, &ap); g_variant_get_va (value, format, NULL, &ap);
va_end (ap); va_end (ap);