client/advertising: Fix not disabling local-name in Includes
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Tue, 10 Aug 2021 21:59:12 +0000 (14:59 -0700)
committerAyush Garg <ayush.garg@samsung.com>
Fri, 11 Mar 2022 13:38:36 +0000 (19:08 +0530)
When setting a advertising.name other than on/off it shall also disable
local-name flag from Includes as that would cause an error when parsed
by the daemon.

Signed-off-by: Anuj Jain <anuj01.jain@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>
client/advertising.c

index b693ba5..ef7aa2e 100644 (file)
@@ -887,6 +887,13 @@ void ad_advertise_local_name(DBusConnection *conn, const char *name)
 
        g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, "LocalName");
 
+       /* Remove local-name from Includes since LocalName would be set */
+       if (ad.name) {
+               ad.name = false;
+               g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE,
+                                                       "Includes");
+       }
+
        return bt_shell_noninteractive_quit(EXIT_SUCCESS);
 }