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
Fix Released
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.

Tags: files
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.