Project

General

Profile

Actions
Copy link

Bug #13981

closed

PHP Error on ``status_interfaces.php`` with empty switch VLAN group configuration and assigned VLAN interfaces

Added by Steve Wheeler 7 months ago. Updated 7 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Jim Pingle
Category:
Interfaces
Target version:
23.05
Start date:
Due date:
% Done:

100%

Estimated time:
Release Notes:
Default
Affected Plus Version:
23.01
Affected Architecture:
SG-2100, SG-3100

Description

Devices with switches are hitting this after upgrading to 23.01.

PHP Errors:
[17-Feb-2023 21:21:44 Europe/Paris] PHP Fatal error:  Uncaught TypeError: Cannot access offset of type string on string in /usr/local/www/status_interfaces.php:137
Stack trace:
#0 {main}
  thrown in /usr/local/www/status_interfaces.php on line 137

Reported in 3100 and 2100

Actions
Copy link
 #1

Updated by Steve Wheeler 7 months ago

This is triggered by the presence of empty <vlangroups> tags in the switch config which are not usually there.

They can end up there if you set the switch in .1q mode, add a vlan group and then set it back to port vlan mode.

Actions
Copy link
 #2

Updated by Marcos M 7 months ago

For reference, the <vlangroups> element in /cf/conf/config.xml can be removed to work around the issue (e.g. by using Diagnostics / Edit File).

Actions
Copy link
 #3

Updated by Jim Pingle 7 months ago

  • Category changed from Web Interface to Interfaces
  • Status changed from New to In Progress
  • Assignee set to Jim Pingle
Actions
Copy link
 #4

Updated by Jim Pingle 7 months ago

To trigger this you also need at least one assigned VLAN tagged interface -- it need not be on the switch interface.

Actions
Copy link
 #5

Updated by Jim Pingle 7 months ago

  • Subject changed from PHP Error: /usr/local/www/status_interfaces.php:137 to PHP Error on ``status_interfaces.php`` with empty switch VLAN group configuration and assigned VLAN interfaces
Actions
Copy link
 #6

Updated by Jim Pingle 7 months ago

  • Status changed from In Progress to Feedback
  • % Done changed from 0 to 100

Fix committed to plus tree (5841d5850265476100b719e60e38b65887cd1460).

You can install the System Patches package and then create an entry for the diff below to apply the fix.

diff --git a/src/usr/local/www/status_interfaces.php b/src/usr/local/www/status_interfaces.php
index 77a0071870..2dd1d7fa16 100644
--- a/src/usr/local/www/status_interfaces.php
+++ b/src/usr/local/www/status_interfaces.php
@@ -134,8 +134,8 @@ foreach ($ifdescrs as $ifdescr => $ifname):

     $ifhwinfo = $ifinfo['hwif'];
     $vlan = interface_is_vlan($ifinfo['hwif']);
-    if ($vlan && is_array($config['switches']['switch'][0]['vlangroups']['vlangroup'])) {
-        foreach ($config['switches']['switch'][0]['vlangroups']['vlangroup'] as $vlangroup) {
+    if ($vlan) {
+        foreach (config_get_path('switches/switch/0/vlangroups/vlangroup', []) as $vlangroup) {
             if ($vlangroup['vlanid'] == $vlan['tag']) {
                 $ifhwinfo .= ', switchports: ' . $vlangroup['members'];
                 break;
Actions
Copy link
 #7

Updated by Danilo Zrenjanin 7 months ago

  • Status changed from Feedback to Resolved

Replicated the issue on SG-2100.

23.01-RELEASE (amd64)
built on Fri Feb 10 20:06:33 UTC 2023
FreeBSD 14.0-CURRENT

The patch fixes it. I am marking the ticket as resovled.

Actions
Copy link

Also available in: Atom PDF