Fix restore script generation with RETAIN_ALIASES=Yes

git-svn-id: https://shorewall.svn.sourceforge.net/svnroot/shorewall/trunk@3185 fbd18981-670d-0410-9b5c-8dc0c1a9a2bb

Shorewall/changelog.txt

@@ -10,6 +10,8 @@ Changes in 3.0.4
 
 5)  Fix bug in NONE intra-zone policy handling.
 
+6)  Fix bug in RETAIN_ALIASES=Yes.
+
 Changes in 3.0.3
 
 1) Implement "shorewall show macros"

Shorewall/firewall

@@ -3021,7 +3021,8 @@ setup_nat() {
     do_one_nat() {
 	local add_ip_aliases=$ADD_IP_ALIASES iface=${interface%:*}
 
-	if [ -n "$add_ip_aliases" ]; then
+	if [ -n "$
+	_aliases" ]; then
 	    case $interface in
 		*:)
 		    interface=${interface%:}
@@ -7581,7 +7582,11 @@ add_ip_aliases()
 
 	shift;shift
 
-	list_search $external $(find_interface_addresses $interface) || do_one
+	if list_search $external $(find_interface_addresses $interface) ; then
+	    save_command qt ip addr add ${external}$(address_details) dev $interface $label
+	else
+	    do_one
+	fi
     done
 }
 

Shorewall/releasenotes.txt

@@ -60,6 +60,16 @@ Problems Corrected in 3.0.4
 3)  Previously, an intra-zone policy of NONE would cause a startup error. That
     problem has been corrected.
 
+4)  When RETAIN_ALIASES=Yes, the script produced by "shorewall save" did not
+    add the retained aliases. This means that the following sequence of
+    events resulted in missing aliases:
+
+	    shorewall start
+	    shorewall restart
+	    shorewall save
+	    reboot
+	    shorewall -f start (which is the default during boot up)
+
 New Features in 3.0.4
 
 1)  By popular demand, the 'Limit' action described at