diff options
author | Ermal LUÇI <eri@pfsense.org> | 2014-11-26 20:34:12 +0100 |
---|---|---|
committer | Ermal LUÇI <eri@pfsense.org> | 2014-11-26 20:34:28 +0100 |
commit | 5e0a3256b291cc54d14dcb16c1b07018e1febd41 (patch) | |
tree | d16da1bc191af1b00ec0994f0d3a12d9f2d4e724 /usr | |
parent | 4f5577f6628bb6e761127d6202b6d25f14a53821 (diff) | |
download | pfsense-5e0a3256b291cc54d14dcb16c1b07018e1febd41.zip pfsense-5e0a3256b291cc54d14dcb16c1b07018e1febd41.tar.gz |
convert_real_interface_to_friendly_interface_name() goes and checks the parent and this gives wrong information 99.9 percent of the time on scenarios like when this is called for unassigned vlans etc, while its real purpose is just to check if the interface is assigned and return the intermeddiate/config name of the interface. Leave the get_parent_option there in the function but it needs to be asked specifically for.
Diffstat (limited to 'usr')
-rw-r--r-- | usr/local/www/interfaces.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/usr/local/www/interfaces.php b/usr/local/www/interfaces.php index 7d2a066..c044ea7 100644 --- a/usr/local/www/interfaces.php +++ b/usr/local/www/interfaces.php @@ -734,7 +734,7 @@ if ($_POST['apply']) { // Need code to handle MLPPP if we ever use $realhwif for MLPPP handling $parent_realhwif = $realhwif_array[0]; $parent_if = convert_real_interface_to_friendly_interface_name($parent_realhwif); - if (!empty($parent_if) && $parent_if != $if && !empty($config['interfaces'][$parent_if]['mtu'])) { + if (!empty($parent_if) && !empty($config['interfaces'][$parent_if]['mtu'])) { if ($_POST['mtu'] > intval($config['interfaces'][$parent_if]['mtu'])) $input_errors[] = gettext("The MTU of a VLAN cannot be greater than that of its parent interface."); } |