From: Florian Forster <octo@leeloo.lan.home.verplant.org>
Date: Tue, 29 Sep 2009 13:51:07 +0000 (+0200)
Subject: netapp plugin: Fix handling of the SIS value.
X-Git-Tag: collectd-4.9.0~73^2~16
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=a9fa8d97fe0eaa49ea1c5e7a1ca031dd03a84f5f;p=collectd.git

netapp plugin: Fix handling of the SIS value.
---

diff --git a/src/netapp.c b/src/netapp.c
index 83cfe454..57fb59de 100644
--- a/src/netapp.c
+++ b/src/netapp.c
@@ -1236,7 +1236,6 @@ static int cna_handle_volume_usage_data (const char *hostname, /* {{{ */
 		na_elem_t *sis;
 		const char *sis_state;
 		uint64_t sis_saved_reported;
-		uint64_t sis_saved;
 
 		volume_name = na_child_get_string (elem_volume, "name");
 		if (volume_name == NULL)
@@ -1293,7 +1292,7 @@ static int cna_handle_volume_usage_data (const char *hostname, /* {{{ */
 		/* size-saved is actually a 32 bit number, so ... time for some guesswork. */
 		if ((sis_saved_reported >> 32) != 0) {
 			/* In case they ever fix this bug. */
-			sis_saved = sis_saved_reported;
+			v->sis_saved = sis_saved_reported;
 		} else { /* really hacky work-around code. {{{ */
 			uint64_t sis_saved_percent;
 			uint64_t sis_saved_guess;
@@ -1328,14 +1327,14 @@ static int cna_handle_volume_usage_data (const char *hostname, /* {{{ */
 
 			if (sis_saved_guess < guess2) {
 				if ((sis_saved_guess - guess1) < (guess2 - sis_saved_guess))
-					sis_saved = guess1;
+					v->sis_saved = guess1;
 				else
-					sis_saved = guess2;
+					v->sis_saved = guess2;
 			} else {
 				if ((sis_saved_guess - guess2) < (guess3 - sis_saved_guess))
-					sis_saved = guess2;
+					v->sis_saved = guess2;
 				else
-					sis_saved = guess3;
+					v->sis_saved = guess3;
 			}
 		} /* }}} end of 32-bit workaround */
 	} /* for (elem_volume) */