Project

General

Profile

Actions

Bug #22030

closed

Authentication Bypass in frontend user authentication (sysext:saltedpasswords)

Added by Marcus Krause almost 15 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
-
Target version:
-
Start date:
2010-01-30
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.3
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Knowing a salted hashed password is sufficient to authenticate a Frontend user when using activated sytem extension saltedpassword in the frontend.

Copy a salted hashed password from a FE user record. Make a login attempt with username and this copied salted password hash.

Due to regularily dealing with SQL Injection issues, we obviously want to prevent such bypassing.

Reported by Sven Haertwig

Vulnerable TYPO3 Core versions: 4.3.0, 4.3.1

TYPO3 Security OTRS X-Reference: #2010013010000011
(issue imported from #M13372)


Files

0013372.diff (2.72 KB) 0013372.diff Administrator Admin, 2010-01-30 12:17
0013372-v2.diff (2.26 KB) 0013372-v2.diff Administrator Admin, 2010-02-01 16:59

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #51941: Exclusive FE usage flag in configuration has no effectClosed2013-09-12

Actions
Related to TYPO3 Core - Bug #73673: Service chaining impossible with SaltedPasswordServiceClosed2016-02-25

Actions
Actions #1

Updated by Marcus Krause almost 15 years ago

(Initial) patch introduces a new variable in saltedpassword's service class that overwrites extension configuration variable "onlyAuthService".

Using the extension configuration variable "onlyAuthService", you can decide if further authentication methods should be tried in case authentication fails.

The new to be introduced behaviour:
If we clearly identify a salted hashed password in the database user record and authentication fails, there's no sense to try another authentication method. We are overwriting whatever is set in "onlyAuthService" and return a code "0" that stops further authentication tries.

Actions #2

Updated by Dmitry Dulepov almost 15 years ago

I attached a patch with the renamed variable. I do not insist on renaming but I think it is more clear this way.

+1 to the patch by testing and reading.

Actions #3

Updated by Dmitry Dulepov almost 15 years ago

Hm. Mantis renamed my file to just "v2.diff" (stripped bug number).

Actions #4

Updated by Oliver Hader over 14 years ago

Committed to SVN
  • TYPO3_4-3 (rev. 6980)
  • Trunk (rev. 6979)
Actions #5

Updated by Helmut Hummel over 8 years ago

  • Project changed from 1716 to TYPO3 Core
  • Description updated (diff)
  • Category deleted (Communication)
  • Target version deleted (-1)
  • Is Regression set to No
Actions

Also available in: Atom PDF