diff options
| author | Bastien Nocera <hadess@hadess.net> | 2021-02-19 12:23:08 +0100 |
|---|---|---|
| committer | Bastien Nocera <hadess@hadess.net> | 2021-02-19 13:10:34 +0100 |
| commit | 5f9f962c4d67f7bdde3b53e402028bbc65319b20 (patch) | |
| tree | 3162315f281dfc25bf9fa4646df3b32f11453395 | |
| parent | 7e33b001605b13c0155e7fe3b8fe2a723c92c56d (diff) | |
| download | gnome-bluetooth-5f9f962c4d67f7bdde3b53e402028bbc65319b20.tar.gz | |
lib: Add guards to helper functions
Clear warnings are better than crashes.
| -rw-r--r-- | lib/bluetooth-client.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/lib/bluetooth-client.c b/lib/bluetooth-client.c index 5a149353..607da321 100644 --- a/lib/bluetooth-client.c +++ b/lib/bluetooth-client.c @@ -163,9 +163,10 @@ compare_address (GtkTreeStore *store, static gboolean get_iter_from_path (GtkTreeStore *store, - GtkTreeIter *iter, - const char *path) + GtkTreeIter *iter, + const char *path) { + g_return_val_if_fail (path != NULL, FALSE); return iter_search(store, iter, NULL, compare_path, (gpointer) path); } @@ -174,6 +175,7 @@ get_iter_from_proxy(GtkTreeStore *store, GtkTreeIter *iter, GDBusProxy *proxy) { + g_return_val_if_fail (proxy != NULL, FALSE); return iter_search(store, iter, NULL, compare_path, (gpointer) g_dbus_proxy_get_object_path (proxy)); } @@ -186,6 +188,9 @@ get_iter_from_address (GtkTreeStore *store, { GtkTreeIter parent_iter; + g_return_val_if_fail (address != NULL, FALSE); + g_return_val_if_fail (adapter != NULL, FALSE); + if (get_iter_from_proxy (store, &parent_iter, adapter) == FALSE) return FALSE; |