diff options
| author | David Zeuthen <davidz@redhat.com> | 2010-08-09 13:15:58 -0400 |
|---|---|---|
| committer | David Zeuthen <davidz@redhat.com> | 2010-08-09 13:15:58 -0400 |
| commit | 07186ab99e66aba27a10b43f834d04fbdee302ca (patch) | |
| tree | dc66923be16eebaf84572e306d4afcd32c9959e4 /src/polkit/polkitsubject.c | |
| parent | eab94aa83209559b2c3f48490778177f1f3b6f97 (diff) | |
| download | polkit-07186ab99e66aba27a10b43f834d04fbdee302ca.tar.gz | |
Add g_return_if_fail() to all public API entry points
Signed-off-by: David Zeuthen <davidz@redhat.com>
Diffstat (limited to 'src/polkit/polkitsubject.c')
| -rw-r--r-- | src/polkit/polkitsubject.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/src/polkit/polkitsubject.c b/src/polkit/polkitsubject.c index 99d49cd..40ef2e9 100644 --- a/src/polkit/polkitsubject.c +++ b/src/polkit/polkitsubject.c @@ -86,6 +86,7 @@ polkit_subject_get_type (void) guint polkit_subject_hash (PolkitSubject *subject) { + g_return_val_if_fail (POLKIT_IS_SUBJECT (subject), 0); return POLKIT_SUBJECT_GET_IFACE (subject)->hash (subject); } @@ -104,6 +105,9 @@ gboolean polkit_subject_equal (PolkitSubject *a, PolkitSubject *b) { + g_return_val_if_fail (POLKIT_IS_SUBJECT (a), FALSE); + g_return_val_if_fail (POLKIT_IS_SUBJECT (b), FALSE); + if (!g_type_is_a (G_TYPE_FROM_INSTANCE (a), G_TYPE_FROM_INSTANCE (b))) return FALSE; @@ -122,6 +126,7 @@ polkit_subject_equal (PolkitSubject *a, gchar * polkit_subject_to_string (PolkitSubject *subject) { + g_return_val_if_fail (POLKIT_IS_SUBJECT (subject), NULL); return POLKIT_SUBJECT_GET_IFACE (subject)->to_string (subject); } @@ -144,6 +149,8 @@ polkit_subject_exists (PolkitSubject *subject, GAsyncReadyCallback callback, gpointer user_data) { + g_return_if_fail (POLKIT_IS_SUBJECT (subject)); + g_return_if_fail (cancellable == NULL || G_IS_CANCELLABLE (cancellable)); POLKIT_SUBJECT_GET_IFACE (subject)->exists (subject, cancellable, callback, @@ -165,6 +172,9 @@ polkit_subject_exists_finish (PolkitSubject *subject, GAsyncResult *res, GError **error) { + g_return_val_if_fail (POLKIT_IS_SUBJECT (subject), FALSE); + g_return_val_if_fail (G_IS_ASYNC_RESULT (res), FALSE); + g_return_val_if_fail (error == NULL || *error == NULL, FALSE); return POLKIT_SUBJECT_GET_IFACE (subject)->exists_finish (subject, res, error); @@ -188,6 +198,9 @@ polkit_subject_exists_sync (PolkitSubject *subject, GCancellable *cancellable, GError **error) { + g_return_val_if_fail (POLKIT_IS_SUBJECT (subject), FALSE); + g_return_val_if_fail (cancellable == NULL || G_IS_CANCELLABLE (cancellable), FALSE); + g_return_val_if_fail (error == NULL || *error == NULL, FALSE); return POLKIT_SUBJECT_GET_IFACE (subject)->exists_sync (subject, cancellable, error); @@ -213,6 +226,7 @@ polkit_subject_from_string (const gchar *str, gchar *endptr; g_return_val_if_fail (str != NULL, NULL); + g_return_val_if_fail (error == NULL || *error == NULL, NULL); /* TODO: we could do something with VFuncs like in g_icon_from_string() */ |