mgmtd: fix a couple compilation warnings.

Also an empty (thus non-replace) notify selectors message shouldn't
clear the selectors, it should just do nothing.

Signed-off-by: Christian Hopps <chopps@labn.net>

2 files changed, 2 insertions, 3 deletions

diff --git a/lib/mgmt_be_client.c b/lib/mgmt_be_client.c
index 6e2fb05e84..49879f3f53 100644
--- a/lib/mgmt_be_client.c
+++ b/lib/mgmt_be_client.c
@@ -1061,7 +1061,7 @@ static void be_client_handle_notify(struct mgmt_be_client *client, void *msgbuf,
 	struct mgmt_msg_notify_data *notif_msg = msgbuf;
 	struct nb_node *nb_node;
 	struct lyd_node *dnode;
-	const char *data;
+	const char *data = NULL;
 	const char *notif;
 	LY_ERR err;
 
diff --git a/mgmtd/mgmt_fe_adapter.c b/mgmtd/mgmt_fe_adapter.c
index 5f53c928a4..09d1910cee 100644
--- a/mgmtd/mgmt_fe_adapter.c
+++ b/mgmtd/mgmt_fe_adapter.c
@@ -1518,7 +1518,6 @@ static void fe_adapter_handle_notify_select(struct mgmt_fe_session_ctx *session,
 	const char **selectors = NULL;
 	const char **new;
 
-	/* An empty message clears the selectors */
 	if (msg_len >= sizeof(*msg)) {
 		selectors = mgmt_msg_native_strings_decode(msg, msg_len,
 							   msg->selectors);
@@ -1531,7 +1530,7 @@ static void fe_adapter_handle_notify_select(struct mgmt_fe_session_ctx *session,
 	if (msg->replace) {
 		darr_free_free(session->notify_xpaths);
 		session->notify_xpaths = selectors;
-	} else {
+	} else if (selectors) {
 		new = darr_append_nz(session->notify_xpaths,
 				     darr_len(selectors));
 		memcpy(new, selectors, darr_len(selectors) * sizeof(*selectors));