[GNUnet-SVN] [gnunet] branch master updated: fix crash on monitor disconnect

[gnunet]

This is an automated email from the git hooks/post-receive script.

grothoff pushed a commit to branch master
in repository gnunet.

The following commit(s) were added to refs/heads/master by this push:
     new d3245124b fix crash on monitor disconnect
d3245124b is described below

commit d3245124b4b2918c6dbd64aa237e96d96279e48b
Author: Christian Grothoff <address@hidden>
AuthorDate: Thu May 10 11:35:02 2018 +0200

    fix crash on monitor disconnect
---
 src/namestore/gnunet-service-namestore.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/namestore/gnunet-service-namestore.c 
b/src/namestore/gnunet-service-namestore.c
index a92b8104a..ffc76a911 100644
--- a/src/namestore/gnunet-service-namestore.c
+++ b/src/namestore/gnunet-service-namestore.c
@@ -918,6 +918,8 @@ client_disconnect_cb (void *cls,
              client);
   for (struct ZoneMonitor *zm = monitor_head; NULL != zm; zm = zm->next)
   {
+    struct StoreActivity *san;
+
     if (nc != zm->nc)
       continue;
     GNUNET_CONTAINER_DLL_remove (monitor_head,
@@ -928,11 +930,13 @@ client_disconnect_cb (void *cls,
       GNUNET_SCHEDULER_cancel (zm->task);
       zm->task = NULL;
     }
-    for (struct StoreActivity *sa = sa_head; NULL != sa; sa = sa->next)
+    for (struct StoreActivity *sa = sa_head; NULL != sa; sa = san)
     {
+      san = sa->next;
       if (zm == sa->zm_pos)
       {
         sa->zm_pos = zm->next;
+        /* this may free sa */
         continue_store_activity (sa);
       }
     }
@@ -943,6 +947,7 @@ client_disconnect_cb (void *cls,
   {
     if (sa->nc == nc)
     {
+      /* this may free sa */
       free_store_activity (sa);
       break; /* there can only be one per nc */
     }

-- 
To stop receiving notification emails like this one, please contact
address@hidden