From 0fa791b29d4b3aa2154f322dda5a7c0d2caa1ede Mon Sep 17 00:00:00 2001 From: =?utf8?q?Javier=20Jard=C3=B3n?= Date: Thu, 22 Apr 2010 03:07:49 +0200 Subject: [PATCH] [gsettings] Add g_return_*_if_fail() guards Reported by Christian Persch here: https://bugzilla.gnome.org/show_bug.cgi?id=616405 --- gio/gsettings.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/gio/gsettings.c b/gio/gsettings.c index 210a73c..19bb680 100644 --- a/gio/gsettings.c +++ b/gio/gsettings.c @@ -393,6 +393,8 @@ g_settings_init (GSettings *settings) void g_settings_delay (GSettings *settings) { + g_return_if_fail (G_IS_SETTINGS (settings)); + if (settings->priv->delayed) return; @@ -496,6 +498,8 @@ g_settings_set_property (GObject *object, gboolean g_settings_get_has_unapplied (GSettings *settings) { + g_return_val_if_fail (G_IS_SETTINGS (settings), FALSE); + return settings->priv->delayed && g_delayed_settings_backend_get_has_unapplied ( G_DELAYED_SETTINGS_BACKEND (settings->priv->backend)); @@ -736,6 +740,8 @@ g_settings_get_value (GSettings *settings, GVariant *sval; gchar *path; + g_return_val_if_fail (G_IS_SETTINGS (settings), NULL); + sval = g_settings_schema_get_value (settings->priv->schema, key, &options); if G_UNLIKELY (sval == NULL) @@ -842,6 +848,8 @@ g_settings_set_value (GSettings *settings, GVariant *sval; gchar *path; + g_return_val_if_fail (G_IS_SETTINGS (settings), FALSE); + sval = g_settings_schema_get_value (settings->priv->schema, key, NULL); correct_type = g_variant_is_of_type (value, g_variant_get_type (sval)); g_variant_unref (sval); @@ -945,6 +953,8 @@ g_settings_is_writable (GSettings *settings, gboolean writable; gchar *path; + g_return_val_if_fail (G_IS_SETTINGS (settings), FALSE); + path = g_strconcat (settings->priv->path, name, NULL); writable = g_settings_backend_get_writable (settings->priv->backend, path); g_free (path); @@ -976,6 +986,8 @@ g_settings_get_child (GSettings *settings, gchar *child_name; GSettings *child; + g_return_val_if_fail (G_IS_SETTINGS (settings), NULL); + child_name = g_strconcat (name, "/", NULL); child_schema = g_settings_schema_get_value (settings->priv->schema, child_name, NULL); @@ -1304,6 +1316,8 @@ g_settings_bind_with_mapping (GSettings *settings, gchar *detailed_signal; GQuark binding_quark; + g_return_if_fail (G_IS_SETTINGS (settings)); + objectclass = G_OBJECT_GET_CLASS (object); binding = g_slice_new0 (GSettingsBinding); @@ -1495,6 +1509,8 @@ g_settings_bind_writable (GSettings *settings, gchar *detailed_signal; GParamSpec *pspec; + g_return_if_fail (G_IS_SETTINGS (settings)); + pspec = g_object_class_find_property (G_OBJECT_GET_CLASS (object), property); if (pspec == NULL) { -- 2.7.4