This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include
, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | |||
3 | namespace yii2mod\rbac\models; |
||
4 | |||
5 | use Yii; |
||
6 | use yii\base\BaseObject; |
||
7 | use yii\base\InvalidConfigException; |
||
8 | use yii\web\IdentityInterface; |
||
9 | |||
10 | /** |
||
11 | * Class AssignmentModel |
||
12 | * |
||
13 | * @package yii2mod\rbac\models |
||
14 | */ |
||
15 | class AssignmentModel extends BaseObject |
||
16 | { |
||
17 | /** |
||
18 | * @var IdentityInterface |
||
19 | */ |
||
20 | public $user; |
||
21 | |||
22 | /** |
||
23 | * @var int User id |
||
24 | */ |
||
25 | public $userId; |
||
26 | |||
27 | /** |
||
28 | * @var \yii\rbac\ManagerInterface |
||
29 | */ |
||
30 | protected $manager; |
||
31 | |||
32 | /** |
||
33 | * AssignmentModel constructor. |
||
34 | * |
||
35 | * @param IdentityInterface $user |
||
36 | * @param array $config |
||
37 | * |
||
38 | * @throws InvalidConfigException |
||
39 | */ |
||
40 | public function __construct(IdentityInterface $user, $config = []) |
||
41 | { |
||
42 | $this->user = $user; |
||
43 | $this->userId = $user->getId(); |
||
0 ignored issues
–
show
|
|||
44 | $this->manager = Yii::$app->authManager; |
||
45 | |||
46 | if ($this->userId === null) { |
||
47 | throw new InvalidConfigException('The "userId" property must be set.'); |
||
48 | } |
||
49 | |||
50 | parent::__construct($config); |
||
51 | } |
||
52 | |||
53 | /** |
||
54 | * Assign a roles and permissions to the user. |
||
55 | * |
||
56 | * @param array $items |
||
57 | * |
||
58 | * @return bool |
||
59 | */ |
||
60 | View Code Duplication | public function assign(array $items): bool |
|
0 ignored issues
–
show
This method seems to be duplicated in your project.
Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository.
Loading history...
|
|||
61 | { |
||
62 | foreach ($items as $name) { |
||
63 | $item = $this->manager->getRole($name); |
||
0 ignored issues
–
show
Are you sure the assignment to
$item is correct as $this->manager->getRole($name) (which targets yii\rbac\ManagerInterface::getRole() ) seems to always return null.
This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A
{
function getObject()
{
return null;
}
}
$a = new A();
$object = $a->getObject();
The method The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.
Loading history...
|
|||
64 | $item = $item ?: $this->manager->getPermission($name); |
||
65 | $this->manager->assign($item, $this->userId); |
||
0 ignored issues
–
show
It seems like
$item defined by $item ?: $this->manager->getPermission($name) on line 64 can be null ; however, yii\rbac\ManagerInterface::assign() does not accept null , maybe add an additional type check?
Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass|null */
function mayReturnNull() { }
function doesNotAcceptNull(stdClass $x) { }
// With potential error.
function withoutCheck() {
$x = mayReturnNull();
doesNotAcceptNull($x); // Potential error here.
}
// Safe - Alternative 1
function withCheck1() {
$x = mayReturnNull();
if ( ! $x instanceof stdClass) {
throw new \LogicException('$x must be defined.');
}
doesNotAcceptNull($x);
}
// Safe - Alternative 2
function withCheck2() {
$x = mayReturnNull();
if ($x instanceof stdClass) {
doesNotAcceptNull($x);
}
}
Loading history...
|
|||
66 | } |
||
67 | |||
68 | return true; |
||
69 | } |
||
70 | |||
71 | /** |
||
72 | * Revokes a roles and permissions from the user. |
||
73 | * |
||
74 | * @param array $items |
||
75 | * |
||
76 | * @return bool |
||
77 | */ |
||
78 | View Code Duplication | public function revoke(array $items): bool |
|
0 ignored issues
–
show
This method seems to be duplicated in your project.
Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository.
Loading history...
|
|||
79 | { |
||
80 | foreach ($items as $name) { |
||
81 | $item = $this->manager->getRole($name); |
||
0 ignored issues
–
show
Are you sure the assignment to
$item is correct as $this->manager->getRole($name) (which targets yii\rbac\ManagerInterface::getRole() ) seems to always return null.
This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A
{
function getObject()
{
return null;
}
}
$a = new A();
$object = $a->getObject();
The method The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.
Loading history...
|
|||
82 | $item = $item ?: $this->manager->getPermission($name); |
||
83 | $this->manager->revoke($item, $this->userId); |
||
0 ignored issues
–
show
It seems like
$item defined by $item ?: $this->manager->getPermission($name) on line 82 can be null ; however, yii\rbac\ManagerInterface::revoke() does not accept null , maybe add an additional type check?
Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass|null */
function mayReturnNull() { }
function doesNotAcceptNull(stdClass $x) { }
// With potential error.
function withoutCheck() {
$x = mayReturnNull();
doesNotAcceptNull($x); // Potential error here.
}
// Safe - Alternative 1
function withCheck1() {
$x = mayReturnNull();
if ( ! $x instanceof stdClass) {
throw new \LogicException('$x must be defined.');
}
doesNotAcceptNull($x);
}
// Safe - Alternative 2
function withCheck2() {
$x = mayReturnNull();
if ($x instanceof stdClass) {
doesNotAcceptNull($x);
}
}
Loading history...
|
|||
84 | } |
||
85 | |||
86 | return true; |
||
87 | } |
||
88 | |||
89 | /** |
||
90 | * Get all available and assigned roles and permissions |
||
91 | * |
||
92 | * @return array |
||
93 | */ |
||
94 | public function getItems(): array |
||
95 | { |
||
96 | $available = []; |
||
97 | $assigned = []; |
||
98 | |||
99 | foreach (array_keys($this->manager->getRoles()) as $name) { |
||
100 | $available[$name] = 'role'; |
||
101 | } |
||
102 | |||
103 | View Code Duplication | foreach (array_keys($this->manager->getPermissions()) as $name) { |
|
0 ignored issues
–
show
This code seems to be duplicated across your project.
Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository.
Loading history...
|
|||
104 | if ($name[0] != '/') { |
||
105 | $available[$name] = 'permission'; |
||
106 | } |
||
107 | } |
||
108 | |||
109 | foreach ($this->manager->getAssignments($this->userId) as $item) { |
||
110 | $assigned[$item->roleName] = $available[$item->roleName]; |
||
111 | unset($available[$item->roleName]); |
||
112 | } |
||
113 | |||
114 | return [ |
||
115 | 'available' => $available, |
||
116 | 'assigned' => $assigned, |
||
117 | ]; |
||
118 | } |
||
119 | } |
||
120 |
Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly.
For example, imagine you have a variable
$accountId
that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to theid
property of an instance of theAccount
class. This class holds a proper account, so the id value must no longer be false.Either this assignment is in error or a type check should be added for that assignment.