avoid chinese finger trap logic, put it all on one side

Author: busterb

lib/msf/ui/console/command_dispatcher/auxiliary.rb

@@ -96,9 +96,10 @@ def cmd_run(*args)
     }
 
     # Always run passive modules in the background
-    if (mod.passive || mod.passive_action?(action || mod.default_action))
+    if mod.is_a?(Msf::Module::HasActions) &&
+        (mod.passive || mod.passive_action?(action || mod.default_action))
       jobify = true
-    end if mod.is_a?(Msf::Module::HasActions)
+    end
 
     begin
       mod.run_simple(