Bug #34620

Wrong return value for getAccessDecisionVoters()

Added by Bernhard Fischer over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Must have
Category:
Security
Start date:
2012-03-06
Due date:
% Done:

100%

PHP Version:
Has patch:
No
Complexity:
no-brainer

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

Revision 94a07994 (diff)
Added by Bernhard Fischer over 7 years ago

[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

Revision 7e055f0b (diff)
Added by Bernhard Fischer over 7 years ago

[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 7 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 7 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 7 years ago

  • Assignee set to Bernhard Fischer
  • Target version set to 1.0.4

#4 Updated by Gerrit Code Review over 7 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 7 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 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF