From: Florian Forster <octo@noris.net>
Date: Wed, 21 Apr 2010 14:19:53 +0000 (+0200)
Subject: network plugin: Don't access values in an already freed array.
X-Git-Tag: collectd-4.8.5~2
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=99ea6c890424d5045cd33db8daa7c28d887b2a88;p=collectd.git

network plugin: Don't access values in an already freed array.
---

diff --git a/src/network.c b/src/network.c
index c34eb38e..054dc88b 100644
--- a/src/network.c
+++ b/src/network.c
@@ -720,11 +720,11 @@ static int parse_part_values (void **ret_buffer, size_t *ret_buffer_len,
 		    break;
 
 		  default:
-		    sfree (pkg_types);
-		    sfree (pkg_values);
 		    NOTICE ("network plugin: parse_part_values: "
 			"Don't know how to handle data source type %"PRIu8,
 			pkg_types[i]);
+		    sfree (pkg_types);
+		    sfree (pkg_values);
 		    return (-1);
 		} /* switch (pkg_types[i]) */
 	}