webui: Stop stripping html tags from snmp creds (#7951)

This commit is contained in:
Neil Lathwood
2017-12-25 15:51:31 +00:00
committed by Tony Murray
parent f75079ab47
commit 27368a2d98
2 changed files with 12 additions and 7 deletions

View File

@@ -52,7 +52,7 @@ if ($hostname !== false) {
); );
} elseif ($_POST['snmpver'] === 'v2c' || $_POST['snmpver'] === 'v1') { } elseif ($_POST['snmpver'] === 'v2c' || $_POST['snmpver'] === 'v1') {
if ($_POST['community']) { if ($_POST['community']) {
$config['snmp']['community'] = array(clean($_POST['community'])); $config['snmp']['community'] = array(clean($_POST['community'], false));
} }
$snmpver = clean($_POST['snmpver']); $snmpver = clean($_POST['snmpver']);
@@ -60,11 +60,11 @@ if ($hostname !== false) {
} elseif ($_POST['snmpver'] === 'v3') { } elseif ($_POST['snmpver'] === 'v3') {
$v3 = array( $v3 = array(
'authlevel' => clean($_POST['authlevel']), 'authlevel' => clean($_POST['authlevel']),
'authname' => clean($_POST['authname']), 'authname' => clean($_POST['authname'], false),
'authpass' => clean($_POST['authpass']), 'authpass' => clean($_POST['authpass'], false),
'authalgo' => clean($_POST['authalgo']), 'authalgo' => clean($_POST['authalgo']),
'cryptopass' => clean($_POST['cryptopass']), 'cryptopass' => clean($_POST['cryptopass'], false),
'cryptoalgo' => clean($_POST['cryptoalgo']), 'cryptoalgo' => clean($_POST['cryptoalgo'], false),
); );
array_push($config['snmp']['v3'], $v3); array_push($config['snmp']['v3'], $v3);

View File

@@ -1513,11 +1513,16 @@ function print_list($list, $format, $max = 10)
/** /**
* @param $value * @param $value
* @param bool $strip_tags
* @return string * @return string
*/ */
function clean($value) function clean($value, $strip_tags = true)
{ {
if ($strip_tags === true) {
return strip_tags(mres($value)); return strip_tags(mres($value));
} else {
return mres($value);
}
} }
/** /**