From 91b840d5d2ae0d274c3b7b105d2a76bef2209598 Mon Sep 17 00:00:00 2001 From: Franco Fichtner Date: Fri, 24 Jun 2022 13:17:02 +0200 Subject: [PATCH] interfaces: inconsistencies in wireless handling #5825 --- src/www/interfaces.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/www/interfaces.php b/src/www/interfaces.php index 62983d1f9..3c0e6d57b 100644 --- a/src/www/interfaces.php +++ b/src/www/interfaces.php @@ -520,7 +520,7 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { $pconfig['wpa_strict_rekey'] = isset($a_interfaces[$if]['wireless']['wpa']['wpa_strict_rekey']); $pconfig['wpa_enable'] = isset($a_interfaces[$if]['wireless']['wpa']['enable']); } - if (is_array($a_interfaces[$if]['wireless']['wep']) && is_array($a_interfaces[$if]['wireless']['wep']['key'])) { + if (!empty($a_interfaces[$if]['wireless']['wep']['key'])) { $i = 1; foreach ($a_interfaces[$if]['wireless']['wep']['key'] as $wepkey) { $pconfig['key' . $i] = $wepkey['value']; @@ -982,8 +982,8 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { $new_ppp_config = array(); // copy physical interface data (wireless is a strange case, partly managed via interface_sync_wireless_clones) - $new_config["if"] = $old_config["if"]; - if (!empty($old_config['wireless'])) { + $new_config['if'] = $old_config['if']; + if (isset($old_config['wireless'])) { $new_config['wireless'] = $old_config['wireless']; } //