commit 59ccb368c479946fc60f5dc1dc1883f809654823
author Michal Vasko <mvasko@cesnet.cz> Thu Nov 05 10:21:44 2015 +0100
committer Michal Vasko <mvasko@cesnet.cz> Thu Nov 05 10:21:44 2015 +0100
tree ea9df5472c50a4fc09de8e13377e44d6488c0c34
parent b0f6c092e2445d6d5ccd81838b71cdd65f0c9862

BUGFIX break the loop if locked_session is NULL

src/mod_netconf.c

diff --git a/src/mod_netconf.c b/src/mod_netconf.c
index ccd62e8..af50aae 100644
--- a/src/mod_netconf.c
+++ b/src/mod_netconf.c
@@ -479,7 +479,7 @@
 	}
 	/* remove session from the active sessions list -> nobody new can now work with session */
 	for (locked_session = netconf_sessions_list;
-         strcmp(nc_session_get_id(locked_session->session), session_id);
+         locked_session && strcmp(nc_session_get_id(locked_session->session), session_id);
          locked_session = locked_session->next);
 
     if (!locked_session) {
@@ -647,7 +647,7 @@
 	}
 	/* get session where send the RPC */
 	for (locked_session = netconf_sessions_list;
-         strcmp(nc_session_get_id(locked_session->session), session_id);
+         locked_session && strcmp(nc_session_get_id(locked_session->session), session_id);
          locked_session = locked_session->next);
 	if (locked_session != NULL) {
 		session = locked_session->session;
@@ -1571,7 +1571,7 @@
 	}
 
 	for (locked_session = netconf_sessions_list;
-         strcmp(nc_session_get_id(locked_session->session), session_id);
+         locked_session && strcmp(nc_session_get_id(locked_session->session), session_id);
          locked_session = locked_session->next);
 	if ((locked_session != NULL) && (locked_session->hello_message != NULL)) {
 		DEBUG("LOCK mutex %s", __func__);
@@ -1619,7 +1619,7 @@
 	}
 
 	for (locked_session = netconf_sessions_list;
-         strcmp(nc_session_get_id(locked_session->session), session_id);
+         locked_session && strcmp(nc_session_get_id(locked_session->session), session_id);
          locked_session = locked_session->next);
 	if (locked_session != NULL) {
 		DEBUG("LOCK mutex %s", __func__);
@@ -1713,7 +1713,7 @@
 	}
 
 	for (locked_session = netconf_sessions_list;
-         strcmp(nc_session_get_id(locked_session->session), session_id);
+         locked_session && strcmp(nc_session_get_id(locked_session->session), session_id);
          locked_session = locked_session->next);
 	if (locked_session != NULL) {
 		DEBUG("LOCK mutex %s", __func__);

src/notification-server.c

diff --git a/src/notification-server.c b/src/notification-server.c
index 8c15e21..e3e6461 100644
--- a/src/notification-server.c
+++ b/src/notification-server.c
@@ -271,7 +271,7 @@
 		return (NULL);
 	}
 	for (locked_session = netconf_sessions_list;
-         strcmp(nc_session_get_id(locked_session->session), session_id);
+         locked_session && strcmp(nc_session_get_id(locked_session->session), session_id);
          locked_session = locked_session->next);
 	return locked_session;
 }