Remko Lodder <[email protected]> wrote
  in <[email protected]>:

re>
re> On Oct 11, 2013, at 4:12 PM, Steve Wills <[email protected]> wrote:
re>
re> > I'm having the same issue.
re>
re> I am working with Hiroki to resolve this. So far his work seems to fix
re> this, he will commit that as soon as we can reasonable think
re> that this is OK (and it looks that way :))

 Let me summarize what I noticed so far:

  (1) A jail with jail_fdescfs_enable="YES" fails to start.

  (2) A jail with multiple parameters in jail_parameters="..." fails
      to start.

 (1) was fixed in r256385.  The attached patch should fix (2).  Please
 try it if you still have a problem.

 Jail configurations can be complex, so there may be some more cases
 which break the old behaviors.  Please send me your report if you
 have differences between before and after the rc.d/jail change.

-- Hiroki
Index: etc/rc.d/jail
===================================================================
--- etc/rc.d/jail	(revision 256385)
+++ etc/rc.d/jail	(working copy)
@@ -94,7 +94,7 @@
 #
 parse_options()
 {
-	local _j
+	local _j _p
 	_j=$1

 	_confwarn=0
@@ -234,7 +234,9 @@
 			    "\"procfs ${_rootdir%/}/proc procfs rw 0 0\";"
 		fi

-		echo "	${_parameters};"
+		for _p in $_parameters; do
+			echo "	${_p%\;};"
+		done

 		eval : \${jail_${_j}_mount_enable:=${jail_mount_enable:-NO}}
 		if checkyesno jail_${_j}_mount_enable; then

Attachment: pgppWFigxE8N9.pgp
Description: PGP signature

Reply via email to