gatt: proxy_property_changed: check for NULL iterator

The passed iterator can be NULL as in
src/gatt-database.c::properties_changed():
...
   proxy->prop_func(..., ..., iter=NULL, ...)
   +--client/gatt.c::property_changed_cb(..., ..., iter, ...);
      +--dbus_message_iter_get_arg_type(iter);
...

Signed-off-by: Manika Shrivastava <manika.sh@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>

diff --git a/src/gatt-database.c b/src/gatt-database.c
index 465d279..6982d2d 100644 (file)
--- a/src/gatt-database.c
+++ b/src/gatt-database.c
@@ -3505,17 +3505,19 @@ static void property_changed_cb(GDBusProxy *proxy, const char *name,
        if (strcmp(name, "Value"))
                return;
 
-       if (dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_ARRAY) {
-               DBG("Malformed \"Value\" property received");
-               return;
-       }
+       if (iter) {
+               if (dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_ARRAY) {
+                       DBG("Malformed \"Value\" property received");
+                       return;
+               }
 
-       dbus_message_iter_recurse(iter, &array);
-       dbus_message_iter_get_fixed_array(&array, &value, &len);
+               dbus_message_iter_recurse(iter, &array);
+               dbus_message_iter_get_fixed_array(&array, &value, &len);
 
-       if (len < 0) {
-               DBG("Malformed \"Value\" property received");
-               return;
+               if (len < 0) {
+                       DBG("Malformed \"Value\" property received");
+                       return;
+               }
        }
 
        /* Truncate the value if it's too large */