malhuda/developer
1
0
Fork 0
mirror of https://github.com/easy-wi/developer.git synced 2025-02-20 11:23:28 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fixing suhosin check 2

Browse Source 
When using empty($max) the follwowing two expression need to be switched.
If Suhosin is not installed @ini_get('suhosin.post.max_vars') and @ini_get('suhosin.request.max_vars') will return false. empty($max) will return true and count($dnsarray) should be set as $max.
...Took me only one year to notice I missed that change here.
This commit is contained in:
GrillGuth 2017-09-13 01:03:53 +02:00 committed by GitHub
parent 698e46837c
commit 6f765606b8
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
stuff/admin/voice_tsdns.php
View File

@ -234,7 +234,7 @@ if ($ui->w('action',4, 'post') and !token(true)) {
$maxRequests = @ini_get('suhosin.request.max_vars');
$i = 0;
$max = ($maxRequests and $maxPost and $maxPost<$maxRequests) ? $maxPost : $maxRequests;
$max = (empty($max)) ? ($max-10)/7 : count($dnsarray);
$max = (empty($max)) ? count($dnsarray) : ($max-10)/7;
$dnsarray = tsdns('li', $ssh2ip, $ssh2port, $ssh2user, $publickey, $keyname, $ssh2password, 'N', $serverdir, $bit, array(''), array(''), array(''), $reseller_id);
@ -305,7 +305,7 @@ if ($ui->w('action',4, 'post') and !token(true)) {
$maxRequests = @ini_get('suhosin.request.max_vars');
$i = 0;
$max = ($maxRequests and $maxPost and $maxPost<$maxRequests) ? $maxPost : $maxRequests;
$max = (empty($max)) ? ($max-10)/7 : count($dnsarray);
$max = (empty($max)) ? count($dnsarray) : ($max-10)/7;
$query = $sql->prepare("SELECT `id`,`cname`,`vname`,`name` FROM `userdata` WHERE `resellerid`=? AND `accounttype`='u' ORDER BY `id` DESC");
$query->execute(array($reseller_id));
@ -479,4 +479,4 @@ if ($ui->w('action',4, 'post') and !token(true)) {
configureDateTables('-1', '1, "asc"', 'ajax.php?w=datatable&d=tsdnsmasterserver');
$template_file = 'admin_voice_tsdns_list.tpl';
}
}