Use fail_with() instead of return

2019-04-02 13:42:07 -04:00 · 2019-04-02 13:42:07 -04:00 · f353df952c
parent 0069eed4e2
commit f353df952c
1 changed files with 2 additions and 4 deletions
--- a/modules/auxiliary/gather/rails_doubletap_file_read.rb
+++ b/modules/auxiliary/gather/rails_doubletap_file_read.rb
@ -90,10 +90,8 @@ class MetasploitModule < Msf::Auxiliary
      return
    end

-    unless datastore['TARGET_FILE'][0] == '/'
-      print_error "TARGET_FILE must be an absolute path (eg. /etc/passwd)."
-      return
-    end
+    fail_with(Failure::BadConfig, 'TARGET_FILE must be an absolute path (eg. /etc/passwd).') unless datastore['TARGET_FILE'][0] == '/'
+

    print_status "Requesting file #{datastore['TARGET_FILE']}"