Project

General

Profile

Actions
Copy link

Bug #2490

closed
JP

Reassigning interfaces on console during restore causes them to be disabled, lose descriptions

Bug #2490: Reassigning interfaces on console during restore causes them to be disabled, lose descriptions

Added by Jim Pingle over 14 years ago. Updated about 14 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Interfaces
Target version:
2.1
Start date:
06/11/2012
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.1
Affected Architecture:

Description

When restoring a config, and reassigning interfaces at the console, somehow this happens:
  • LAN/WAN are OK
  • OPT interfaces are disabled
  • OPT interfaces have lost their descriptions (custom names)

The latter two are quite bad situations to land in, especially if you manage the GUI from one of those OPT interfaces.
(This happens on current 2.1 snapshots, but may be an even older problem)

In the config history it has quite an unhelpful description, "(system): made unknown change".

Config diff when this happens:

$ diff -u /conf/backup/config-1339430243.xml /conf/backup/config-1339430374.xml
--- /conf/backup/config-1339430243.xml    2012-06-11 11:59:34.000000000 -0400
+++ /conf/backup/config-1339430374.xml    2012-06-11 12:43:19.000000000 -0400
@@ -102,11 +102,10 @@
             <gatewayv6>WANGWv6</gatewayv6>
         </wan>
         <opt1>
-            <descr><![CDATA[DMZ]]></descr>
+            <descr><![CDATA[OPT1]]></descr>
             <if>em1</if>
             <bridge/>
             <spoofmac/>
-            <enable/>
             <disableftpproxy/>
             <ipaddr>68.76.213.194</ipaddr>
             <subnet>26</subnet>
@@ -115,7 +114,7 @@
             <dhcp6-pd-sla-id>none</dhcp6-pd-sla-id>
         </opt1>
         <opt2>
-            <descr><![CDATA[SYNC]]></descr>
+            <descr><![CDATA[OPT2]]></descr>
             <if>fxp1</if>
             <bridge/>
             <ipaddr>192.168.150.15</ipaddr>
@@ -123,7 +122,6 @@
             <gateway/>
             <spoofmac/>
             <mtu/>
-            <enable/>
             <disableftpproxy/>
         </opt2>
     </interfaces>
@@ -2225,9 +2223,9 @@
         <tab/>
     </installedpackages>
     <revision>
-        <description><![CDATA[admin@192.168.x.x: /interfaces_assign.php made unknown change]]></description>
-        <time>1339430354</time>
-        <username>admin@192.168.x.x</username>
+        <description><![CDATA[(system):  made unknown change]]></description>
+        <time>1339430374</time>
+        <username>(system)</username>
     </revision>
     <virtualip>
         <vip>
@@ -2703,5 +2701,4 @@
         <synchronizednsforwarder>on</synchronizednsforwarder>
         <synchronizecaptiveportal>on</synchronizecaptiveportal>
     </hasync>
-    <dhcpdv6/>
 </pfsense>

EL Updated by Ermal Luçi over 14 years ago Actions
Copy link
 #1

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

Applied in changeset commit:62784b050ad874bd8544531954b4b158036f9a23.

EF Updated by Erik Fonnesbeck over 14 years ago Actions
Copy link
 #2

Is there any reason to ever unset the "enable" setting there? Is not the "enable" setting only set for opt interfaces when there is an existing configuration? The code looks like afterward it removes the opt interfaces you did not configure anyway, so I do not see any point in doing this.

EL Updated by Ermal Luçi over 14 years ago Actions
Copy link
 #3

If descr is set the enable will not be removed.
For historic reason i left it there!

Please move on.

EF Updated by Erik Fonnesbeck over 14 years ago Actions
Copy link
 #4

The reason I asked is that interfaces.php lets you store a blank descr in the config to get the default name.

CB Updated by Chris Buechler over 14 years ago Actions
Copy link
 #5

I don't see a reason to ever unset enable there, why is that there? Is that only if you re-assign interfaces and don't assign the OPTs then it disables them? In that case it's fine, short of that I don't see a reason to disable.

JP Updated by Jim Pingle about 14 years ago Actions
Copy link
 #6

  • Status changed from Feedback to Resolved
Actions
Copy link

Also available in: Atom