From 9a9740dbfe3e8afa80b27e6109d44d317fc27d1a Mon Sep 17 00:00:00 2001 From: Franco Fichtner Date: Mon, 4 Feb 2019 12:38:01 +0000 Subject: [PATCH] mvc: style issues --- .../library/OPNsense/OpenVPN/PlainOpenVPN.php | 2 +- .../OPNsense/Base/FieldTypes/PortField.php | 50 ++++++++++--------- 2 files changed, 27 insertions(+), 25 deletions(-) diff --git a/src/opnsense/mvc/app/library/OPNsense/OpenVPN/PlainOpenVPN.php b/src/opnsense/mvc/app/library/OPNsense/OpenVPN/PlainOpenVPN.php index d1cd6ace6..2c61a4dd1 100644 --- a/src/opnsense/mvc/app/library/OPNsense/OpenVPN/PlainOpenVPN.php +++ b/src/opnsense/mvc/app/library/OPNsense/OpenVPN/PlainOpenVPN.php @@ -124,7 +124,7 @@ class PlainOpenVPN extends BaseExporter implements IExportProvider if (!empty($tmp_subject)) { $tmp_subject .= ", "; } - $tmp_subject .= "{$key}={$value}" ; + $tmp_subject .= "{$key}={$value}"; } $conf[] = "verify-x509-name \"{$tmp_subject}\" subject"; if (!empty($this->config['server_cert_is_srv'])) { diff --git a/src/opnsense/mvc/app/models/OPNsense/Base/FieldTypes/PortField.php b/src/opnsense/mvc/app/models/OPNsense/Base/FieldTypes/PortField.php index 4f535f1e9..7c04b66b0 100644 --- a/src/opnsense/mvc/app/models/OPNsense/Base/FieldTypes/PortField.php +++ b/src/opnsense/mvc/app/models/OPNsense/Base/FieldTypes/PortField.php @@ -1,32 +1,31 @@ array("min_range"=>1, "max_range"=>65535))) === false) { + if (filter_var( + $port, + FILTER_VALIDATE_INT, + array('options' => array('min_range' => 1, 'max_range' => 65535)) + ) === false) { $messages[] = $this->getValidationMessage(); break; }