Security: Multiple URL: Fix vulnerability whereby a user could access a session was assigned to another campus in which the user was not registered

Author: @juan-cortizas-ponte
pull/4335/head^2
Juan Cortizas Ponte 3 years ago committed by GitHub
parent 54316d5f4e
commit 3c5b82bc8a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 10
      main/session/about.php

@ -29,6 +29,16 @@ $session = api_get_session_entity($sessionId);
if (!$session) {
api_not_allowed(true);
}
if (api_is_multiple_url_enabled()) {
$accessUrlId = api_get_current_access_url_id();
$sessionOnUrl = UrlManager::relation_url_session_exist($sessionId, $accessUrlId);
if (!$sessionOnUrl) {
api_not_allowed(true);
}
}
$htmlHeadXtra[] = api_get_asset('readmore-js/readmore.js');
$courses = [];
$sessionCourses = $em->getRepository('ChamiloCoreBundle:Session')->getCoursesOrderedByPosition($session);

Loading…
Cancel
Save