Revert module loading behavoir

mythril/analysis/security.py

@@ -2,6 +2,7 @@ from collections import defaultdict
 from ethereum.opcodes import opcodes
 from mythril.analysis import modules
 import pkgutil
+import importlib.util
 import logging
 
 
@@ -28,16 +29,25 @@ def get_detection_module_hooks():
     return dict(hook_dict)
 
 
-def get_detection_modules(entrypoint, except_modules=()):
+def get_detection_modules(entrypoint, include_modules=()):
-    except_modules = list(except_modules) + ["base"]
+    include_modules = list(include_modules)
+
     _modules = []
 
-    for loader, name, _ in pkgutil.walk_packages(modules.__path__):
+    if not len(include_modules):
-        module = loader.find_module(name).load_module(name)
+
-        if (
+        for loader, name, _ in pkgutil.walk_packages(modules.__path__):
-            module.__name__ not in except_modules
+            module = loader.find_module(name).load_module(name)
-            and module.detector.entrypoint == entrypoint
+            if (
-        ):
+                module.__name__ !=  "base"
+                and module.detector.entrypoint == entrypoint
+            ):
+                _modules.append(module)
+
+    else:
+        for module_name in include_modules:
+            module = importlib.import_module(module_name, modules)
+
             _modules.append(module)
 
     logging.info("Found %s detection modules", len(_modules))
@@ -48,7 +58,7 @@ def fire_lasers(statespace, module_names=()):
     logging.info("Starting analysis")
 
     issues = []
-    for module in get_detection_modules(entrypoint="post", except_modules=module_names):
+    for module in get_detection_modules(entrypoint="post", include_modules=module_names):
         logging.info("Executing " + module.detector.name)
         issues += module.detector.execute(statespace)
 

mythril/interfaces/cli.py

@@ -398,24 +398,27 @@ def main():
                     exit_with_error(args.outform, "Error saving graph: " + str(e))
 
             else:
-                report = mythril.fire_lasers(
+                try:
-                    strategy=args.strategy,
+                    report = mythril.fire_lasers(
-                    address=address,
+                        strategy=args.strategy,
-                    modules=[m.strip() for m in args.modules.strip().split(",")]
+                        address=address,
-                    if args.modules
+                        modules=[m.strip() for m in args.modules.strip().split(",")]
-                    else [],
+                        if args.modules
-                    verbose_report=args.verbose_report,
+                        else [],
-                    max_depth=args.max_depth,
+                        verbose_report=args.verbose_report,
-                    execution_timeout=args.execution_timeout,
+                        max_depth=args.max_depth,
-                    create_timeout=args.create_timeout,
+                        execution_timeout=args.execution_timeout,
-                    max_transaction_count=args.max_transaction_count,
+                        create_timeout=args.create_timeout,
-                )
+                        max_transaction_count=args.max_transaction_count,
-                outputs = {
+                    )
-                    "json": report.as_json(),
+                    outputs = {
-                    "text": report.as_text(),
+                        "json": report.as_json(),
-                    "markdown": report.as_markdown(),
+                        "text": report.as_text(),
-                }
+                        "markdown": report.as_markdown(),
-                print(outputs[args.outform])
+                    }
+                    print(outputs[args.outform])
+                except ModuleNotFoundError as e:
+                    exit_with_error(args.outform, "Error loading analyis modules: " + format(e))
 
         elif args.statespace_json: