diff --git a/app/views/layouts/tutorial/broken_auth_sess/_user_pass_enum.html.erb b/app/views/layouts/tutorial/broken_auth_sess/_user_pass_enum.html.erb index 8d79d13..6637631 100644 --- a/app/views/layouts/tutorial/broken_auth_sess/_user_pass_enum.html.erb +++ b/app/views/layouts/tutorial/broken_auth_sess/_user_pass_enum.html.erb @@ -33,7 +33,7 @@
Username and Password Enumeration
-Within /app/models/user.rb:
+
Within /app/models/user.rb:
@@ -53,8 +53,29 @@ return auth end+
On lines 9 and 12 you'll notice that the application generates two error messages.
+Within /app/controllers/sessions_controller.rb:
+
+ def create + + begin + user = User.authenticate(params[:email], params[:password]) + rescue Exception => e + end + + if user + session[:id] = user.id if User.where(:id => user.id).exists? + redirect_to home_dashboard_index_path + else + flash[:error] = e.message + render "new" + end + + end ++
On line 5 you see the exception message object "e" is created. On line 11, the message is displayed.
- On lines 9 and 12 you'll notice that the application generates two error messages. One indicates the email address (username) doesn't exist on the system. The other indicates that the password is incorrect. Although the application will render both error messages, either one of the error messages would be harmful by itself. This type of information can be used by an attacker to harvest email addresses or usernames. Once that list is gathered, passwords can be guessed for each account. If the username being enumerated is actually an email address, a phishing campaign could ensue with emails made to look like they are originating from the vulnerable site. + One of these messages indicates the email address (username) doesn't exist on the system. The other indicates that the password is incorrect. Although the application will render both error messages, either one of the error messages would be harmful by itself. This type of information can be used by an attacker to harvest email addresses or usernames. Once that list is gathered, passwords can be guessed for each account. If the username being enumerated is actually an email address, a phishing campaign could ensue with emails made to look like they are originating from the vulnerable site.