3571e53040
patches for easier mirroring, to eliminate a special copy, to make www.freebsd.org/security a full copy of security.freebsd.org and be eventually be the same. For now files are just sitting there. The symlinks are missing. Discussed on: www (repository location) Discussed with: simon (so)
214 lines
5.5 KiB
Diff
214 lines
5.5 KiB
Diff
Index: etc/rc.d/jail
|
|
===================================================================
|
|
RCS file: /home/ncvs/src/etc/rc.d/jail,v
|
|
retrieving revision 1.23.2.3.2.2
|
|
diff -u -d -r1.23.2.3.2.2 jail
|
|
--- etc/rc.d/jail 7 Jul 2006 07:25:21 -0000 1.23.2.3.2.2
|
|
+++ etc/rc.d/jail 9 Jan 2007 21:53:36 -0000
|
|
@@ -67,6 +67,8 @@
|
|
[ -z "${_fstab}" ] && _fstab="/etc/fstab.${_j}"
|
|
eval _flags=\"\${jail_${_j}_flags:-${jail_flags}}\"
|
|
[ -z "${_flags}" ] && _flags="-l -U root"
|
|
+ eval _consolelog=\"\${jail_${_j}_consolelog:-${jail_consolelog}}\"
|
|
+ [ -z "${_consolelog}" ] && _consolelog="/var/log/jail_${_j}_console.log"
|
|
|
|
# Debugging aid
|
|
#
|
|
@@ -86,6 +88,7 @@
|
|
debug "$_j exec start: $_exec_start"
|
|
debug "$_j exec stop: $_exec_stop"
|
|
debug "$_j flags: $_flags"
|
|
+ debug "$_j consolelog: $_consolelog"
|
|
|
|
if [ -z "${_hostname}" ]; then
|
|
err 3 "$name: No hostname has been defined for ${_j}"
|
|
@@ -126,6 +129,56 @@
|
|
fi
|
|
}
|
|
|
|
+# is_current_mountpoint()
|
|
+# Is the directory mount point for a currently mounted file
|
|
+# system?
|
|
+#
|
|
+is_current_mountpoint()
|
|
+{
|
|
+ local _dir _dir2
|
|
+
|
|
+ _dir=$1
|
|
+
|
|
+ _dir=`echo $_dir | sed -Ee 's#//+#/#g' -e 's#/$##'`
|
|
+ [ ! -d "${_dir}" ] && return 1
|
|
+ _dir2=`df ${_dir} | tail +2 | awk '{ print $6 }'`
|
|
+ [ "${_dir}" = "${_dir2}" ]
|
|
+ return $?
|
|
+}
|
|
+
|
|
+# is_symlinked_mountpoint()
|
|
+# Is a mount point, or any of its parent directories, a symlink?
|
|
+#
|
|
+is_symlinked_mountpoint()
|
|
+{
|
|
+ local _dir
|
|
+
|
|
+ _dir=$1
|
|
+
|
|
+ [ -L "$_dir" ] && return 0
|
|
+ [ "$_dir" = "/" ] && return 1
|
|
+ is_symlinked_mountpoint `dirname $_dir`
|
|
+ return $?
|
|
+}
|
|
+
|
|
+# secure_umount
|
|
+# Try to unmount a mount point without being vulnerable to
|
|
+# symlink attacks.
|
|
+#
|
|
+secure_umount()
|
|
+{
|
|
+ local _dir
|
|
+
|
|
+ _dir=$1
|
|
+
|
|
+ if is_current_mountpoint ${_dir}; then
|
|
+ umount -f ${_dir} >/dev/null 2>&1
|
|
+ else
|
|
+ debug "Nothing mounted on ${_dir} - not unmounting"
|
|
+ fi
|
|
+}
|
|
+
|
|
+
|
|
# jail_umount_fs
|
|
# This function unmounts certain special filesystems in the
|
|
# currently selected jail. The caller must call the init_variables()
|
|
@@ -133,27 +186,65 @@
|
|
#
|
|
jail_umount_fs()
|
|
{
|
|
+ local _device _mountpt _rest
|
|
+
|
|
if checkyesno _fdescfs; then
|
|
if [ -d "${_fdescdir}" ] ; then
|
|
- umount -f ${_fdescdir} >/dev/null 2>&1
|
|
+ secure_umount ${_fdescdir}
|
|
fi
|
|
fi
|
|
if checkyesno _devfs; then
|
|
if [ -d "${_devdir}" ] ; then
|
|
- umount -f ${_devdir} >/dev/null 2>&1
|
|
+ secure_umount ${_devdir}
|
|
fi
|
|
fi
|
|
if checkyesno _procfs; then
|
|
if [ -d "${_procdir}" ] ; then
|
|
- umount -f ${_procdir} >/dev/null 2>&1
|
|
+ secure_umount ${_procdir}
|
|
fi
|
|
fi
|
|
if checkyesno _mount; then
|
|
[ -f "${_fstab}" ] || warn "${_fstab} does not exist"
|
|
- umount -a -F "${_fstab}" >/dev/null 2>&1
|
|
+ tail -r ${_fstab} | while read _device _mountpt _rest; do
|
|
+ case ":${_device}" in
|
|
+ :#* | :)
|
|
+ continue
|
|
+ ;;
|
|
+ esac
|
|
+ secure_umount ${_mountpt}
|
|
+ done
|
|
fi
|
|
}
|
|
|
|
+# jail_mount_fstab()
|
|
+# Mount file systems from a per jail fstab while trying to
|
|
+# secure against symlink attacks at the mount points.
|
|
+#
|
|
+# If we are certain we cannot secure against symlink attacks we
|
|
+# do not mount all of the file systems (since we cannot just not
|
|
+# mount the file system with the problematic mount point).
|
|
+#
|
|
+# The caller must call the init_variables() routine before
|
|
+# calling this one.
|
|
+#
|
|
+jail_mount_fstab()
|
|
+{
|
|
+ local _device _mountpt _rest
|
|
+
|
|
+ while read _device _mountpt _rest; do
|
|
+ case ":${_device}" in
|
|
+ :#* | :)
|
|
+ continue
|
|
+ ;;
|
|
+ esac
|
|
+ if is_symlinked_mountpoint ${_mountpt}; then
|
|
+ warn "${_mountpt} has symlink as parent - not mounting from ${_fstab}"
|
|
+ return
|
|
+ fi
|
|
+ done <${_fstab}
|
|
+ mount -a -F "${_fstab}"
|
|
+}
|
|
+
|
|
jail_start()
|
|
{
|
|
echo -n 'Configuring jails:'
|
|
@@ -183,12 +274,16 @@
|
|
if [ ! -f "${_fstab}" ]; then
|
|
err 3 "$name: ${_fstab} does not exist"
|
|
fi
|
|
- mount -a -F "${_fstab}"
|
|
+ jail_mount_fstab
|
|
fi
|
|
if checkyesno _devfs; then
|
|
# If devfs is already mounted here, skip it.
|
|
df -t devfs "${_devdir}" >/dev/null
|
|
if [ $? -ne 0 ]; then
|
|
+ if is_symlinked_mountpoint ${_devdir}; then
|
|
+ warn "${_devdir} has symlink as parent - not starting jail ${_jail}"
|
|
+ continue
|
|
+ fi
|
|
info "Mounting devfs on ${_devdir}"
|
|
devfs_mount_jail "${_devdir}" ${_ruleset}
|
|
# Transitional symlink for old binaries
|
|
@@ -209,13 +304,21 @@
|
|
# cd "$__pwd"
|
|
fi
|
|
if checkyesno _fdescfs; then
|
|
- info "Mounting fdescfs on ${_fdescdir}"
|
|
- mount -t fdescfs fdesc "${_fdescdir}"
|
|
+ if is_symlinked_mountpoint ${_fdescdir}; then
|
|
+ warn "${_fdescdir} has symlink as parent, not mounting"
|
|
+ else
|
|
+ info "Mounting fdescfs on ${_fdescdir}"
|
|
+ mount -t fdescfs fdesc "${_fdescdir}"
|
|
+ fi
|
|
fi
|
|
if checkyesno _procfs; then
|
|
- info "Mounting procfs onto ${_procdir}"
|
|
- if [ -d "${_procdir}" ] ; then
|
|
- mount -t procfs proc "${_procdir}"
|
|
+ if is_symlinked_mountpoint ${_procdir}; then
|
|
+ warn "${_procdir} has symlink as parent, not mounting"
|
|
+ else
|
|
+ info "Mounting procfs onto ${_procdir}"
|
|
+ if [ -d "${_procdir}" ] ; then
|
|
+ mount -t procfs proc "${_procdir}"
|
|
+ fi
|
|
fi
|
|
fi
|
|
_tmp_jail=${_tmp_dir}/jail.$$
|
|
@@ -223,7 +326,7 @@
|
|
${_ip} ${_exec_start} > ${_tmp_jail} 2>&1
|
|
[ "$?" -eq 0 ] && echo -n " $_hostname"
|
|
_jail_id=$(head -1 ${_tmp_jail})
|
|
- tail +2 ${_tmp_jail} >${_rootdir}/var/log/console.log
|
|
+ tail +2 ${_tmp_jail} >${_consolelog}
|
|
rm -f ${_tmp_jail}
|
|
echo ${_jail_id} > /var/run/jail_${_jail}.id
|
|
done
|
|
@@ -242,7 +345,7 @@
|
|
init_variables $_jail
|
|
if [ -n "${_exec_stop}" ]; then
|
|
eval env -i /usr/sbin/jexec ${_jail_id} ${_exec_stop} \
|
|
- >> ${_rootdir}/var/log/console.log 2>&1
|
|
+ >> ${_consolelog} 2>&1
|
|
fi
|
|
killall -j ${_jail_id} -TERM > /dev/null 2>&1
|
|
sleep 1
|