Re: [PATCH v2] mesh: Make meshctl use bt_shell helpers

[Johan Hedberg <johan.hedberg@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1905 bytes --]

Hi Luiz,

On Tue, Dec 05, 2017, Johan Hedberg wrote:
> On Tue, Dec 05, 2017, Luiz Augusto von Dentz wrote:
> > This makes meshctl use bt_shell to manage the menus and command
> > handling.
> > ---
> >  mesh/config-client.c | 187 ++++++++---------
> >  mesh/config-model.h  |   1 -
> >  mesh/main.c          | 571 +++++++++++++++------------------------------------
> >  mesh/onoff-model.c   |  88 +++-----
> >  mesh/util.c          | 155 +-------------
> >  mesh/util.h          |  16 +-
> >  6 files changed, 278 insertions(+), 740 deletions(-)
> 
> There's still something broken. Did you test this at all?
> 
> If I issue "connect 1 2" right after starting meshctl I get this:
> 
> [meshctl]# connect 1 2
> ==16599== Invalid read of size 1
> ==16599==    at 0x4C32B82: strlen (vg_replace_strmem.c:458)
> ==16599==    by 0x4EA5962: g_strdup (in /usr/lib64/libglib-2.0.so.0.5400.2)
> ==16599==    by 0x4EA7404: g_strdupv (in /usr/lib64/libglib-2.0.so.0.5400.2)
> ==16599==    by 0x40861F: set_scan_filter_uuids.constprop.14 (main.c:1613)
> ==16599==    by 0x4086E2: cmd_connect (main.c:1729)
> ==16599==    by 0x41C8F7: cmd_exec (shell.c:290)
> ==16599==    by 0x41C8F7: menu_exec (shell.c:314)
> ==16599==    by 0x41D1B2: shell_exec (shell.c:326)
> ==16599==    by 0x41D1B2: rl_handler (shell.c:473)
> ==16599==    by 0x55DB8BD: rl_callback_read_char (in /usr/lib64/libreadline.so.7.0)
> ==16599==    by 0x41C4E8: input_read (shell.c:765)
> ==16599==    by 0x41D672: watch_callback (io-glib.c:170)
> ==16599==    by 0x4E86BB6: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.5400.2)
> ==16599==    by 0x4E86F5F: ??? (in /usr/lib64/libglib-2.0.so.0.5400.2)
> ==16599==  Address 0x2 is not stack'd, malloc'd or (recently) free'd
> 
> It looks like the cause might be the new UUID string arrays which are
> expected to be NULL terminated.

Something like the attached patch seems to fix it.

Johan

[-- Attachment #2: filter-fix.diff --]
[-- Type: text/plain, Size: 986 bytes --]

diff --git a/mesh/main.c b/mesh/main.c
index b98a4cf99641..3372ecfb9aa5 100644
--- a/mesh/main.c
+++ b/mesh/main.c
@@ -1625,7 +1625,7 @@ commit:
 static void cmd_scan_unprovisioned_devices(int argc, char *argv[])
 {
 	dbus_bool_t enable;
-	char *filters[] = { MESH_PROV_SVC_UUID };
+	char *filters[2] = { MESH_PROV_SVC_UUID };
 
 	if (parse_argument_on_off(argc, argv, &enable) == FALSE)
 		return;
@@ -1700,7 +1700,7 @@ static void cmd_security(int argc, char *argv[])
 
 static void cmd_connect(int argc, char *argv[])
 {
-	char *filters[] = { MESH_PROXY_SVC_UUID };
+	char *filters[2] = { MESH_PROXY_SVC_UUID };
 
 	if (check_default_ctrl() == FALSE)
 		return;
@@ -1753,7 +1753,7 @@ static void cmd_connect(int argc, char *argv[])
 static void prov_disconn_reply(DBusMessage *message, void *user_data)
 {
 	struct mesh_node *node = user_data;
-	char *filters[] = { MESH_PROXY_SVC_UUID };
+	char *filters[2] = { MESH_PROXY_SVC_UUID };
 	DBusError error;
 
 	dbus_error_init(&error);