diff --git a/mesh/agent.c b/mesh/agent.c
index c6ff118..88ad84d 100644
--- a/mesh/agent.c
+++ b/mesh/agent.c
void mesh_agent_remove(struct mesh_agent *agent)
{
- if (!l_queue_find(agents, simple_match, agent))
+ if (!agent || !l_queue_find(agents, simple_match, agent))
return;
agent_free(agent);
diff --git a/mesh/mesh.c b/mesh/mesh.c
index 8971f7c..a1c26e7 100644
--- a/mesh/mesh.c
+++ b/mesh/mesh.c
l_free(pending);
}
+static void free_pending_join_call(bool failed)
+{
+ if (!join_pending)
+ return;
+
+ if (join_pending->disc_watch)
+ l_dbus_remove_watch(dbus_get_bus(),
+ join_pending->disc_watch);
+
+ acceptor_cancel(&mesh);
+
+ mesh_agent_remove(join_pending->agent);
+
+ if (failed) {
+ storage_remove_node_config(join_pending->node);
+ node_free(join_pending->node);
+ }
+
+ l_free(join_pending);
+ join_pending = NULL;
+}
+
void mesh_cleanup(void)
{
struct l_dbus_message *reply;
mgmt_unref(mgmt_mesh);
if (join_pending) {
+ /* The Join() call failed since it has not been completed */
reply = dbus_error(join_pending->msg, MESH_ERROR_FAILED,
"Failed. Exiting");
l_dbus_send(dbus_get_bus(), reply);
- if (join_pending->disc_watch)
- l_dbus_remove_watch(dbus_get_bus(),
- join_pending->disc_watch);
-
- if (join_pending->node)
- node_free(join_pending->node);
-
- l_free(join_pending);
- join_pending = NULL;
+ free_pending_join_call(true);
}
l_queue_destroy(attach_queue, attach_exit);
}
}
-static void free_pending_join_call(bool failed)
-{
- if (!join_pending)
- return;
-
- if (join_pending->disc_watch)
- l_dbus_remove_watch(dbus_get_bus(),
- join_pending->disc_watch);
-
- mesh_agent_remove(join_pending->agent);
-
- if (failed) {
- storage_remove_node_config(join_pending->node);
- mesh_agent_remove(join_pending->agent);
- }
-
- l_free(join_pending);
- join_pending = NULL;
-}
-
/* This is being called if the app exits unexpectedly */
static void prov_disc_cb(struct l_dbus *bus, void *user_data)
{
if (join_pending->msg)
l_dbus_message_unref(join_pending->msg);
- acceptor_cancel(&mesh);
-
join_pending->disc_watch = 0;
free_pending_join_call(true);
fail:
l_dbus_send(dbus_get_bus(), reply);
- mesh_agent_remove(join_pending->agent);
- l_free(join_pending);
- join_pending = NULL;
+ free_pending_join_call(true);
}
static struct l_dbus_message *join_network_call(struct l_dbus *dbus,
diff --git a/mesh/prov-acceptor.c b/mesh/prov-acceptor.c
index baa3c4d..d983991 100644
--- a/mesh/prov-acceptor.c
+++ b/mesh/prov-acceptor.c
static void acceptor_free(void)
{
+ if (!prov)
+ return;
- if (prov)
- l_timeout_remove(prov->timeout);
+ l_timeout_remove(prov->timeout);
mesh_send_cancel(bec_filter, sizeof(bec_filter));
mesh_send_cancel(&pkt_filter, sizeof(pkt_filter));
diff --git a/mesh/storage.c b/mesh/storage.c
index 57ae88b..3a6614e 100644
--- a/mesh/storage.c
+++ b/mesh/storage.c
struct json_object *jnode;
const char *dir_name;
+ if (!node)
+ return;
+
jnode = node_jconfig_get(node);
if (jnode)
json_object_put(jnode);