src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/yamlLoad.py

branch
eric7
changeset 9221
bf71ee032bb4
parent 9209
b99e7fd55fd3
child 9325
8157eb19aba5
--- a/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/yamlLoad.py	Wed Jul 13 11:16:20 2022 +0200
+++ b/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/yamlLoad.py	Wed Jul 13 14:55:47 2022 +0200
@@ -19,7 +19,7 @@
 def getChecks():
     """
     Public method to get a dictionary with checks handled by this module.
-    
+
     @return dictionary containing checker lists containing checker function and
         list of codes
     @rtype dict
@@ -34,7 +34,7 @@
 def checkYamlLoad(reportError, context, config):
     """
     Function to check for the use of of yaml load functions.
-    
+
     @param reportError function to be used to report errors
     @type func
     @param context security context object
@@ -42,23 +42,19 @@
     @param config dictionary with configuration data
     @type dict
     """
-    imported = context.isModuleImportedExact('yaml')
+    imported = context.isModuleImportedExact("yaml")
     qualname = context.callFunctionNameQual
     if not imported and isinstance(qualname, str):
         return
-    
-    qualnameList = qualname.split('.')
+
+    qualnameList = qualname.split(".")
     func = qualnameList[-1]
-    if all([
-            'yaml' in qualnameList,
-            func == 'load',
-            not context.checkCallArgValue('Loader', 'SafeLoader'),
-            not context.checkCallArgValue('Loader', 'CSafeLoader'),
-    ]):
-        reportError(
-            context.node.lineno - 1,
-            context.node.col_offset,
-            "S506",
-            "M",
-            "H"
-        )
+    if all(
+        [
+            "yaml" in qualnameList,
+            func == "load",
+            not context.checkCallArgValue("Loader", "SafeLoader"),
+            not context.checkCallArgValue("Loader", "CSafeLoader"),
+        ]
+    ):
+        reportError(context.node.lineno - 1, context.node.col_offset, "S506", "M", "H")

eric ide

mercurial