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
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);
