Fix a NULL pointer dereference during shutdown

Signed-off-by: Florian Forster <octo@collectd.org>

diff --git a/src/plugin.c b/src/plugin.c
index 70372342927c9fd2264cb70952fbd308d8317150..942f8bfe50438066adc80b46ac4175c990f9a642 100644 (file)
--- a/src/plugin.c
+++ b/src/plugin.c
@@ -810,10 +810,12 @@ static void stop_write_threads (void) /* {{{ */
 
        pthread_mutex_lock (&write_lock);
        i = 0;
-       for (q = write_queue_head; q != NULL; q = q->next)
+       for (q = write_queue_head; q != NULL; )
        {
+               write_queue_t *q1 = q;
                plugin_value_list_free (q->vl);
-               sfree (q);
+               q = q->next;
+               sfree (q1);
                i++;
        }
        write_queue_head = NULL;