Fixes a bug where the return was getting ignored by each_user_pass for FTP. As a result, all usernames would get tried, instead of retiring a username once a good password was found.

git-svn-id: file:///home/svn/framework3/trunk@10622 4d416f70-5f16-0410-b530-b9f4589650da
2010-10-09 19:03:49 +00:00 · 2010-10-09 19:03:49 +00:00 · fc755f7a7a
parent b99be2dae6
commit fc755f7a7a
1 changed files with 2 additions and 1 deletions
--- a/modules/auxiliary/scanner/ftp/ftp_login.rb
+++ b/modules/auxiliary/scanner/ftp/ftp_login.rb
@ -55,8 +55,9 @@ class Metasploit3 < Msf::Auxiliary
 		if check_banner
 			each_user_pass { |user, pass|
 				next if user.nil? || user.empty?
-				do_login(user,pass)
+				ret = do_login(user,pass)
 				ftp_quit if datastore['SINGLE_SESSION']
+				return ret
 			}
 			check_anonymous
 		else