From 023a985bfc8a627b1e4ccff797fbc33e3d064d6c Mon Sep 17 00:00:00 2001 From: Kevin O'Connor Date: Thu, 14 Jan 2021 22:13:50 -0500 Subject: gcode_macro: Use deepcopy() on get_status() results If a get_status() method returns a mutable object (such as a list or dict) then it would be possible for a gcode command template to incorrectly alter the program's internal state. Perform a deepcopy() operation on all get_status() return results to avoid that. Signed-off-by: Kevin O'Connor --- klippy/extras/save_variables.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'klippy/extras/save_variables.py') diff --git a/klippy/extras/save_variables.py b/klippy/extras/save_variables.py index 6c88e3cf..f15dbc11 100644 --- a/klippy/extras/save_variables.py +++ b/klippy/extras/save_variables.py @@ -57,7 +57,7 @@ class SaveVariables: gcmd.respond_info("Variable Saved") self.loadVariables() def get_status(self, eventtime): - return {'variables': dict(self.allVariables)} + return {'variables': self.allVariables} def load_config(config): return SaveVariables(config) -- cgit v1.2.3-70-g09d2