From ed0e0a20059aeadddd69c4618d35dfb3aca69ba4 Mon Sep 17 00:00:00 2001
From: Jeremy Allison <jra@samba.org>
Date: Wed, 10 Feb 2010 16:23:33 -0800
Subject: [PATCH] Fix bad use when freeing linked list. Todd Stecher (Original
 author) please check !

Jeremy.
---
 source3/modules/perfcount_test.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/source3/modules/perfcount_test.c b/source3/modules/perfcount_test.c
index f9ea817127b..67be4aa9c61 100644
--- a/source3/modules/perfcount_test.c
+++ b/source3/modules/perfcount_test.c
@@ -176,12 +176,14 @@ static void perfcount_test_dump_counters(void)
 
 	DEBUG(0,("#####  Dumping Performance Counters #####\n"));
 
-	for (i=0; i < 256; i++) {
-	       for (head = g_list[i]; head != NULL; head = head->next) {
-		       perfcount_test_dump_counter(head, 0);
-		       SAFE_FREE(DLIST_PREV(head));
-	       }
-	       SAFE_FREE(head);
+	for (i=0; i < MAX_OP; i++) {
+		struct perfcount_test_counter *next;
+		for (head = g_list[i]; head != NULL; head = next) {
+			next = head->next;
+			perfcount_test_dump_counter(head, 0);
+			SAFE_FREE(head);
+		}
+		g_list[i] = NULL;
 	}
 }
 
-- 
2.11.4.GIT