amb: remove a FIXME for type checking.
authorIsmo Puustinen <ismo.puustinen@intel.com>
Fri, 10 Jan 2014 08:23:11 +0000 (10:23 +0200)
committerKrisztian Litkey <krisztian.litkey@intel.com>
Thu, 8 Jan 2015 16:37:12 +0000 (18:37 +0200)
Change-Id: I4436c15407dda7cdb8362b6bd2304af6d1bee8e1

src/plugins/plugin-amb.c

index d90adec..babe9e0 100644 (file)
@@ -796,16 +796,13 @@ static void basic_property_handler(mrp_dbus_msg_t *msg, dbus_property_watch_t *w
         goto error;
     }
 
-#if 0
-    /* FIXME: check the type later */
+    /* check that D-Bus type matches the expected type */
 
-    if (dbus_message_iter_get_arg_type(&variant_iter)
-                        != w->prop.type) {
-        mrp_log_error("amb: argument type %c did not match expected type %c",
-                dbus_message_iter_get_arg_type(&variant_iter), w->prop.type);
+    if (mrp_dbus_msg_arg_type(msg, NULL) != w->prop.type)  {
+        mrp_log_error("AMB: argument type %c did not match expected type %c",
+                mrp_dbus_msg_arg_type(msg, NULL), w->prop.type);
         goto error;
     }
-#endif
 
     switch (w->prop.type) {
         case MRP_DBUS_TYPE_INT32: