Skip to content

Commit 364c056

Browse files
authored
don't apply preventXSS on 'filter' parameters in export and search (#168)
fixes #98
1 parent d09aa72 commit 364c056

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

htdocs/copy.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
$request['remove'] = (get_request('remove') == 'yes') ? true : false;
4242

4343
if ($request['recursive']) {
44-
$filter = get_request('filter','POST',false,'(objectClass=*)');
44+
$filter = get_request('filter','POST',false,'(objectClass=*)',false);
4545

4646
# Build a tree similar to that of the tree browser to give to r_copy_dn
4747
$ldap['tree'] = array();

htdocs/export_form.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
$request['format'] = get_request('format','GET',false,get_line_end_format());
1818
$request['scope'] = get_request('scope','GET',false,'base');
1919
$request['exporter_id'] = get_request('exporter_id','GET',false,'LDIF');
20-
$request['filter'] = get_request('filter','GET',false,'(objectClass=*)');
20+
$request['filter'] = get_request('filter','GET',false,'(objectClass=*)',false);
2121
$request['attr'] = get_request('attributes','GET',false,'*');
2222
$request['sys_attr'] = get_request('sys_attr','GET') ? true: false;
2323

lib/Query.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ public function accept() {
134134
# If this is a custom search, we need to populate are paramters
135135
if ($this->getID() == 'none') {
136136
$bases = get_request('base','REQUEST',false,null);
137-
$query['filter'] = get_request('filter','REQUEST',false,'objectClass=*');
137+
$query['filter'] = get_request('filter','REQUEST',false,'objectClass=*',false);
138138
$query['scope'] = get_request('scope','REQUEST',false,'sub');
139139
$attrs = get_request('display_attrs','REQUEST',false,'');
140140

lib/export_functions.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ public function accept() {
125125
$query = array();
126126
$base = get_request('dn','REQUEST');
127127
$query['baseok'] = true;
128-
$query['filter'] = get_request('filter','REQUEST',false,'objectclass=*');
128+
$query['filter'] = get_request('filter','REQUEST',false,'objectclass=*',false);
129129
$query['scope'] = get_request('scope','REQUEST',false,'base');
130130
$query['deref'] = $_SESSION[APPCONFIG]->getValue('deref','export');
131131
$query['size_limit'] = 0;

0 commit comments

Comments
 (0)