Sanity Changes

Reverted failure behaviour on line 70
Removed a space that prevented line 98 from working as intended

Author: nanomebia

modules/exploits/unix/webapp/sugarcrm_unserialize_exec.rb

@@ -67,7 +67,7 @@ def on_new_session(client)
         client.fs.file.rm(f)
         print_good("#{peer} - #{f} removed to stay ninja")
       rescue
-        fail_with(Failure::Unknown, "#{peer} - Unable to remove #{f}")
+        print_warning("#{peer} - Unable to remove #{f}")
       end
     end
   end
@@ -95,7 +95,7 @@ def exploit
       'data'   => data
     })
 
-    if res.nil? || res.headers['Location'].include? ('action=Login') || res.get_cookies.empty?
+    if res.nil? || res.headers['Location'].include?('action=Login') || res.get_cookies.empty?
       fail_with(Failure::NoAccess, "#{peer} - Login failed with \"#{username}:#{password}\"")
     end