Skip to content

Commit f23e5db

Browse files
sKarki999fwolf-ilias
authored andcommitted
Fix(SOAP): Add missing permission check to prevent moving objects from foreign containers
Signed-off-by: Releasemanager <webmaster@ilias.de>
1 parent b81e30a commit f23e5db

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

components/ILIAS/soap/classes/class.ilSoapObjectAdministration.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1047,6 +1047,10 @@ public function moveObject(string $sid, int $ref_id, int $target_id)
10471047
return $this->raiseError('No valid target given.', 'Client');
10481048
}
10491049

1050+
if(!$rbacsystem->checkAccess('move', $ref_id)) {
1051+
return $this->raiseError("No permission to move object with id: $ref_id", 'Client');
1052+
}
1053+
10501054
// check for trash
10511055
if (ilObject::_isInTrash($ref_id)) {
10521056
return $this->raiseError('Object is trashed.', 'Client');

0 commit comments

Comments
 (0)