bugfixes & fix pylint

crytic · Jan 27, 2025 · af8e5cf · af8e5cf
1 parent 87fb412
commit af8e5cf
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/slither/tools/mutator/mutators/LIR.py b/slither/tools/mutator/mutators/LIR.py
@@ -36,7 +36,7 @@ def _mutate(self) -> Dict:  # pylint: disable=too-many-branches
                     # Get the string
                     start = variable.source_mapping.start
                     stop = start + variable.source_mapping.length
-                    old_str = node.source_mapping.content
+                    old_str = variable.source_mapping.content
                     line_no = variable.node_initialization.source_mapping.lines
                     if not line_no[0] in self.dont_mutate_line:
                         for value in literal_replacements:
@@ -67,7 +67,7 @@ def _mutate(self) -> Dict:  # pylint: disable=too-many-branches
                             literal_replacements.append("-1")
                     start = variable.source_mapping.start
                     stop = start + variable.source_mapping.length
-                    old_str = node.source_mapping.content
+                    old_str = variable.source_mapping.content
                     line_no = variable.source_mapping.lines
                     if not line_no[0] in self.dont_mutate_line:
                         for new_value in literal_replacements:

diff --git a/slither/tools/mutator/mutators/MVIV.py b/slither/tools/mutator/mutators/MVIV.py
@@ -24,7 +24,7 @@ def _mutate(self) -> Dict:
                     # Get the string
                     start = variable.source_mapping.start
                     stop = variable.expression.source_mapping.start
-                    old_str = node.source_mapping.content
+                    old_str = variable.source_mapping.content
                     new_str = old_str[: old_str.find("=")]
                     line_no = variable.node_initialization.source_mapping.lines
                     if not line_no[0] in self.dont_mutate_line:
@@ -43,7 +43,7 @@ def _mutate(self) -> Dict:
                 if variable.initialized and isinstance(variable.expression, Literal):
                     start = variable.source_mapping.start
                     stop = variable.expression.source_mapping.start
-                    old_str = node.source_mapping.content
+                    old_str = variable.source_mapping.content
                     new_str = old_str[: old_str.find("=")]
                     line_no = variable.source_mapping.lines
                     if not line_no[0] in self.dont_mutate_line:

diff --git a/slither/tools/mutator/mutators/abstract_mutator.py b/slither/tools/mutator/mutators/abstract_mutator.py
@@ -77,6 +77,7 @@ def _mutate(self) -> Dict:
     # pylint: disable=too-many-branches
     def mutate(self) -> Tuple[List[int], List[int], List[int]]:
         all_patches: Dict = {}
+        # pylint: disable=broad-exception-caught
         try:
             # call _mutate function from different mutators
             (all_patches) = self._mutate()