Bug #34620
Wrong return value for getAccessDecisionVoters()
| Status: | Resolved | Start date: | 2012-03-06 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | Bernhard Fischer | % Done: | 100% |
|
| Category: | Security | |||
| Target version: | TYPO3 Flow Base Distribution - 1.0.4 | |||
| PHP Version: | Complexity: | no-brainer | ||
| Has patch: | No | Affected Flow version: | FLOW3 1.0.0 |
Description
The return value of the function getAccessDecisionVoters() in the AccessDecisionVoterManager should return a array. It doesn't really matter because this function is never referenced.
Type hinting for $securityContext is missing the leading \ and hinting for $authenticationManager in the PolicyEnforcement is also wrong.
Associated revisions
[BUGFIX] typo in return statement for getAccessDecisionVoters()
Fix return value to array.
Fix type hinting for var $securityContext and $authenticationManager
Change-Id: I75f825602c7c9eea172e30dc0b831b8736a3727c
Fixes: #34620
Releases: 1.0, 1.1
[BUGFIX] typo in return statement for getAccessDecisionVoters()
Fix return value to array.
Fix type hinting for var $securityContext and $authenticationManager
Change-Id: I05223d157b5cb7bd6c736d08a09c9544a909bff8
Fixes: #34620
Releases: 1.0, 1.1
History
#1 Updated by Gerrit Code Review over 3 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9430
#2 Updated by Gerrit Code Review over 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9430
#3 Updated by Karsten Dambekalns over 3 years ago
- Assigned To set to Bernhard Fischer
- Target version set to 1.0.4
#4 Updated by Gerrit Code Review over 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9430
#5 Updated by Gerrit Code Review over 3 years ago
Patch set 1 for branch FLOW3-1.0 has been pushed to the review server.
It is available at http://review.typo3.org/9589
#6 Updated by Bernhard Fischer over 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7e055f0b2c7e2d0f92992afd0c97007b50ef4aac.