deepcopy dicts to avoid passing around references

Martin Walsh · Martin Walsh · commit 148b86ed9ef3 · 2012-10-10T12:13:30.000-05:00
diff --git a/network/netstats/python_modules/netstats.py b/network/netstats/python_modules/netstats.py
@@ -1,6 +1,7 @@
 import sys
 import re
 import time
+import copy
 
 PARAMS = {}
 
@@ -13,7 +14,7 @@
 tcpext_file = "/proc/net/netstat"
 snmp_file = "/proc/net/snmp"
 
-LAST_METRICS = dict(METRICS)
+LAST_METRICS = copy.deepcopy(METRICS)
 METRICS_CACHE_MAX = 5
 
 stats_pos = {} 
@@ -188,10 +189,10 @@ def get_metrics():
 	    if re.match("TcpExt: [0-9]", line):
                 metrics = re.split("\s+", line)
 
-	file.close
+	file.close()
 
         # update cache
-        LAST_METRICS = dict(METRICS)
+        LAST_METRICS = copy.deepcopy(METRICS)
         METRICS = {
             'time': time.time(),
             'tcpext': metrics
@@ -216,7 +217,7 @@ def get_metrics():
                 METRICS['tcp'] = re.split("\s+", line)
 
 
-    	file.close
+    	file.close()
 
     return [METRICS, LAST_METRICS]
 
