=================================================================== RCS file: /cvs/openbsd/fw_update/fw_install.sh,v retrieving revision 1.43 retrieving revision 1.46 diff -u -r1.43 -r1.46 --- openbsd/fw_update/fw_install.sh 2021/12/02 02:56:37 1.43 +++ openbsd/fw_update/fw_install.sh 2021/12/02 03:46:50 1.46 @@ -1,5 +1,5 @@ #!/bin/ksh -# $OpenBSD: fw_install.sh,v 1.43 2021/12/02 02:56:37 afresh1 Exp $ +# $OpenBSD: fw_install.sh,v 1.46 2021/12/02 03:46:50 afresh1 Exp $ set -e # Copyright (c) 2021 Andrew Hewus Fresh @@ -21,16 +21,26 @@ echo "${DESTDIR}/tmp" } -# tmpdir, do_as, unpriv, and unpriv2 are from install.sub +# tmpdir, do_as, and unpriv are from install.sub # modified to use su(1) when not in the installer. +# modified to use mktemp(1) when not in the installer. # Create a temporary directory based on the supplied directory name prefix. tmpdir() { local _i=1 _dir + if [[ -z $1 ]]; then + echo No tmpdir >&2 + exit 1 + fi - until _dir="${1?}.$_i.$RANDOM" && mkdir -- "$_dir" 2>/dev/null; do - ((++_i < 10000)) || return 1 - done + if [[ -e /usr/bin/mktemp ]]; then + _dir=$( /usr/bin/mktemp -d $1 ) + chown _sndio "$_dir" + else + until _dir="${1%-+(X)}.$_i.$RANDOM" && mkdir -- "$_dir" 2>/dev/null; do + ((++_i < 10000)) || return 1 + done + fi echo "$_dir" } @@ -79,10 +89,6 @@ do_as _sndio "$@" } -unpriv2() { - do_as _file "$@" -} - VNAME=${VNAME:-$(sysctl -n kern.osrelease)} VERSION=${VERSION:-"${VNAME%.*}${VNAME#*.}"} FWDIR=${FWDIR:-$VNAME} @@ -144,7 +150,7 @@ # Create a download directory for the firmware and # check that the _sndio user can read files from # it. Otherwise cleanup and skip the filesystem. - if _tmpsrc=$(tmpdir "$_tmpfs/firmware"); then + if _tmpsrc=$(tmpdir "$_tmpfs/firmware-XXXXXXXXX"); then ( >$_tmpsrc/t && $_unpriv cat $_tmpsrc/t @@ -191,8 +197,8 @@ # sha256. Create a flag file in case ftp failed. Firmware # from net is written to the prefetch area. ( $_unpriv ftp -D "$_t" -Vmo - "$_src/$_f" || >/tmp/fail ) | - ( $_srclocal && unpriv2 sha256 -b >/tmp/h || - unpriv2 -f /tmp/h sha256 -bph /tmp/h >"$_tmpsrc/$_f" ) + ( $_srclocal && sha256 -b >/tmp/h || + sha256 -bph /tmp/h >"$_tmpsrc/$_f" ) # Handle failed transfer. if [[ -f /tmp/fail ]]; then