webapp/postfixadmin
1
0
Fork 0
mirror of https://github.com/postfixadmin/postfixadmin.git synced 2025-07-28 11:41:57 +03:00
Code Activity

Fix parameter parsing for '-1'

Browse Source 
'--quota -1' gets parsed as two options ("quota" and "1"), but it's
meant to be "quota => -1".

Make sure '-1' is considered as a value, not as an option.

Also get rid of unset()'ing $params[$i] and (now?) superfluous recursive
calls to __parseParams() to make the code less confusing.
This commit is contained in:
Christian Boltz
2020-05-02 01:17:34 +02:00
parent fc464d6e69
commit 4e9e3db75d
1 changed files with 8 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
scripts/postfixadmin-cli.php
View File

@ -323,26 +323,22 @@ class PostfixAdmin {
private function __parseParams($params) {
$count = count($params);
for ($i = 0; $i < $count; $i++) {
if (isset($params[$i])) {
if ($params[$i] != '' && $params[$i]{0} === '-') {
# if (isset($params[$i])) {
if ($params[$i] != '' && $params[$i]{0} === '-' && $params[$i] != '-1') {
$key = substr($params[$i], 1);
$this->params[$key] = true;
unset($params[$i]);
if (isset($params[++$i])) {
if (isset($params[$i+1])) {
# TODO: ideally we should know if a parameter can / must have a value instead of whitelisting known valid values starting with '-' (probably only bool doesn't need a value)
if ($params[$i]{0} !== '-' or $params[$i] != '-1') {
$this->params[$key] = $params[$i];
unset($params[$i]);
if ($params[$i+1]{0} === '-' && $params[$i+1] != '-1') {
$this->params[$key] = true;
} else {
$i--;
$this->__parseParams($params);
$this->params[$key] = $params[$i+1];
$i++;
}
}
} else {
$this->args[] = $params[$i];
unset($params[$i]);
}
}
# }
}
}