Modify

Opened 10 years ago

Closed 10 years ago

#238 closed defect (fixed)

Failsafe netmsg on WAN interface clutters upstream network(s)

Reported by: anonymous Owned by: nbd
Priority: normal Milestone: 1.0-rc5
Component: base system Version: 1.0
Keywords: Cc:

Description

Hi all,

The recent change with netmsg and b44 driver introduces subtle problems for upstream networks when the box is reloaded/restarted while connected upstream.
In preinit, the roboswitch is loaded with a flat network, where all interfaces including the WAN interface are on vlan0. Then several messages are sent from 192.168.1.1/24 to broadcast on all interfaces. This causes following problems:

  • Upstream devices learn the MAC of the box, which is not necessarily the correct MAC
    • If port security is enabled, the upstream port may become locked down
  • Upstream devices suffer from MAC database instability
    • All boxes have the same IP address by default
    • Upstream monitoring/watchdogs may get confused and/or may trigger action
  • All upstream devices receive unnecessary broadcast traffic
  • Upstream IP configuration may be in conflict with the 192.168.1.0/24 net
    • An upstream gateway might become hijacked due to overlapping IP segments

The solution is simply to remove the WAN port from the preinit configuration (see patch below).

--
Greetings Bertho

===================================================================
--- package/base-files/default/etc/preinit      (revision 3032)
+++ package/base-files/default/etc/preinit      (working copy)
@@ -15,7 +15,9 @@
        v1p="$(cat /proc/switch/eth0/vlan/1/ports)"
        v2p="$(cat /proc/switch/eth0/vlan/2/ports)"
        echo 1 > /proc/switch/eth0/reset
-       echo "0 1 2 3 4 5u*" > /proc/switch/eth0/vlan/0/ports
+#      echo "0 1 2 3 4 5u*" > /proc/switch/eth0/vlan/0/ports
+# Do not use the WAN port because it clutters upstream networks
+       echo "1 2 3 4 5u*" > /proc/switch/eth0/vlan/0/ports
 }

 # this delay is apparently necessary because the link isn't up yet

Attachments (0)

Change History (3)

comment:1 Changed 10 years ago by nbd

  • Owner changed from developers to nbd
  • Status changed from new to assigned

comment:2 Changed 10 years ago by nbd

  • Milestone set to 1.0-rc5
  • Priority changed from high to normal
  • Version set to 1.0

comment:3 Changed 10 years ago by nbd

  • Resolution set to fixed
  • Status changed from assigned to closed

fixed in [3083]

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.