Project

General

Profile

Actions

Defect #9239

closed

autenticity_token is not checked properly

Added by Karel Pičman over 12 years ago. Updated over 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Accounts / authentication
Target version:
-
Start date:
2011-09-13
Due date:
% Done:

100%

Estimated time:
Resolution:
Fixed
Affected version:

Description

I'm afraid that auhtenticity_token is not checked properly. E.g. While submitting a user change I stop the submitting using Tamper Data Firefox plugin and change authenticity_token or remove the value of authenticity_token completely. To check the Redmine behaviour I change form value admin = 1 in Tamper Data. Althought authenticity_token is not equal to the original value sent from the server, the request is successfully processed. The user got Administrator role and only after that is the user logged out because of wrong authenticity_token.

I'd expect this procedure:
  1. Check authenticity_token
  2. If it's OK then process the request
  3. If not then deny to process the request

I've tried to document the test on attached screen shots:

Step 1: Edit a user
Step 2: Stop execution and alter sent data(admin flag and authenticity_token)
Step 3About your application's environment
Ruby version 1.8.7 (x86_64-linux)
RubyGems version 1.3.7
Rack version 1.1.2
Rails version 2.3.11
Active Record version 2.3.11
Active Resource version 2.3.11
Action Mailer version 2.3.11
Active Support version 2.3.11
Edge Rails revision unknown
Application root /home/picman/tmp/redmine-1.2.1
Environment production
Database adapter mysql
Database schema version 20110511000000: Submit the request
Step 4: We can see, that the user was promoted to administrator.

Clear installation without any changes or additional plugins. The same problem in version 1.2.0.


Files

step_1.png (116 KB) step_1.png Step 1 Karel Pičman, 2011-09-13 14:22
step_2.png (67.8 KB) step_2.png Step 2 Karel Pičman, 2011-09-13 14:22
step_3.png (47.6 KB) step_3.png Step 3 Karel Pičman, 2011-09-13 14:22
step_4.png (115 KB) step_4.png Step 4 Karel Pičman, 2011-09-13 14:22

Related issues

Related to Redmine - Defect #4825: Several related bugs relating to registration, sign in and account preferences.New2010-02-13

Actions
Actions #1

Updated by Karel Pičman over 12 years ago

Is there any chance that somebody will look at this issue? I'm afraid that it has impact on security of each Redmine instance installed on the Internet.

Actions #2

Updated by Felix Schäfer over 12 years ago

So you're logged in as an admin and can change things with a wrong authenticity token, or what seems to be the problem?

Actions #3

Updated by Karel Pičman over 12 years ago

Yes. The user profile is updated first and only after that is the authenticity token checked. The authenticity token must be checked always first otherwise authenticity principle is useless in my opinion.
In case of wrong authenticity token no data can be changed.

Actions #4

Updated by Felix Schäfer over 12 years ago

I think it's been corrected in trunk http://www.redmine.org/projects/redmine/repository/diff?rev=6316&rev_to=6314 , can't find any issue for it though. Could you try it on trunk?

Actions #5

Updated by Felix Schäfer over 12 years ago

(for the record, it's been fixed and released in Chiliproject about 2 months ago in 2.1.0 and 1.5.1 </shameless plug>)

Actions #6

Updated by Karel Pičman over 12 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Works fine in the current trunk. Thank you.

Actions #7

Updated by Etienne Massip over 12 years ago

  • Status changed from Resolved to Closed
  • Resolution set to Fixed

Thanks for your feedback.

Actions

Also available in: Atom PDF