maxuploadsize should consider that post_max_size is the whole POST content

Bug #634829 reported by Ruslan Kabalin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Medium
Ruslan Kabalin

Bug Description

In addition to respecting quota (), maxupload variable (which used to be in filebrowser pieform, now it is in get_max_upload_size function, see 7b2772a03 on master) considers minimum value of either 'post_max_size' or 'upload_max_filesize':

$maxuploadsize = min(get_real_size(ini_get('post_max_size')), get_real_size(ini_get('upload_max_filesize')));

It should be taken into account that post_max_size PHP setting evaluates the whole POST content (which includes file name and other from values, not only the file itself), thus assuming that file should be equal to this value while determining maxupload size is incorrect.

Changed in mahara:
assignee: nobody → Ruslan Kabalin (ruslan-kabalin)
importance: Low → Medium
Changed in mahara:
milestone: none → 1.4.0
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers