bn8/ldapsaisie
1
0
Fork 0
mirror of https://gitlab.easter-eggs.com/ee/ldapsaisie.git synced 2024-11-18 16:19:06 +01:00
Code Issues Projects Releases Wiki Activity

LSsearch::setParamsFromRequest(): add check on parameters load from request data

Browse source
This commit is contained in:
Benjamin Renard 2021-02-02 20:42:02 +01:00
parent eb8cfbf66a
commit 841b3d0bcf
1 changed files with 59 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

62
src/includes/class/class.LSsearch.php
View file

@ -558,10 +558,66 @@ class LSsearch extends LSlog_staticLoggerClass {
/** /**
* Define search parameters by reading request data ($_REQUEST) * Define search parameters by reading request data ($_REQUEST)
* *
* @retval void * @retval boolean True if all parameters found in request data are handled, False otherwise
*/ */
public function setParamsFromRequest() { public function setParamsFromRequest() {
$data = $_REQUEST; $allowedParams = array(
'pattern', 'approx', 'recursive', 'extraDisplayedColumns', 'nbObjectsByPage',
'attributes', 'sortBy', 'sortDirection', 'withoutCache', 'predefinedFilter',
// Following parameters are allowed from request but need additional checks
'filter', 'basedn', 'subDn', 'scope', 'attributes', 'displayFormat',
);
foreach($_REQUEST as $key => $value) {
if (!in_array($key, $allowedParams))
continue;
switch($key) {
case 'filter':
// Parse filter to extract attribute name and check if user have read access on it
// FIXME: Net_LDAP2_Filter could only permit to extract attribute name from a simple
// filter (without sub-filters), so we currently only access this type of filter.
$filter = Net_LDAP2_Filter::parse($value);
if (Net_LDAP2::isError($filter)) {
LSerror :: addErrorCode('LSsearch_03', 'filter');
return;
}
$filter_parts = $filter -> getComponents();
if (!is_array($filter_parts) || !LSsession :: canAccess($this -> LSobject, null, 'r', $filter_parts[0])) {
LSerror :: addErrorCode('LSsearch_03', 'filter');
return;
}
return;
break;
case 'basedn':
case 'subDn':
case 'scope':
// These parameters could only be used if combined with onlyAccessible==True
$data['onlyAccessible'] = True;
break;
case 'attributes':
foreach(ensureIsArray($value) as $attr) {
if (!is_string($attr)) {
LSerror :: addErrorCode('LSsearch_06');
return;
}
if (!LSsession :: canAccess($this -> LSobject, null, 'r', $attr)) {
LSerror :: addErrorCode('LSsearch_11', $attr);
return;
}
}
break;
case 'displayFormat':
if (!LSsession :: canComputeLSformat($value, $this -> LSobject)) {
LSerror :: addErrorCode('LSsearch_11', 'displayFormat');
return;
}
break;
}
$data[$key] = $value;
}
if (self::formIsSubmited()) { if (self::formIsSubmited()) {
// Recursive // Recursive
@ -585,7 +641,7 @@ class LSsearch extends LSlog_staticLoggerClass {
} }
} }
$this -> setParams($data); return $this -> setParams($data);
} }
/** /**