misc/108988: [patch] RELENG_6_2 rc.d/jail unaliases incorrectly, on failed startup

[Kevin Way]

>Number:         108988
>Category:       misc
>Synopsis:       [patch] RELENG_6_2 rc.d/jail unaliases incorrectly, on failed startup
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Feb 09 20:20:06 GMT 2007
>Closed-Date:
>Last-Modified:
>Originator:     Kevin Way
>Release:        RELENG_6_2
>Organization:
InsideSystems, Inc.
>Environment:
FreeBSD bluefin.insidesystems.net 6.2-RELEASE FreeBSD 6.2-RELEASE #0: Fri Jan 12 08:43:30 UTC 2007     root at portnoy.cse.buffalo.edu:/usr/obj/usr/src/sys/SMP  amd64
>Description:
If a jail has an error while setting up, rc.d/jail can end up removing the wrong IP address, becausese it calls ifconfig -alias with an incorrect variable.
>How-To-Repeat:
setup a jail that fails on startup.
>Fix:
--- jail.orig   Fri Feb  9 15:00:20 2007
+++ jail        Fri Feb  9 15:02:59 2007
@@ -331,8 +331,8 @@
                                echo ${_jail_id} > /var/run/jail_${_jail}.id
                        else
                                jail_umount_fs
-                               if [ -n "${jail_interface}" ]; then
-                                       ifconfig ${jail_interface} -alias ${jail_ip}
+                               if [ -n "${_interface}" ]; then
+                                       ifconfig ${_interface} -alias ${_ip}
                                fi
                                echo " cannot start jail \"${_jail}\": "
                                tail +2 ${_tmp_jail}

>Release-Note:
>Audit-Trail:
>Unformatted: