Remove retry loop crutch around disk passphrase prompt
authorkn <kn@openbsd.org>
Mon, 21 Aug 2023 08:59:18 +0000 (08:59 +0000)
committerkn <kn@openbsd.org>
Mon, 21 Aug 2023 08:59:18 +0000 (08:59 +0000)
bioctl(8) now retries itself on mismatch so the installer continues until
the passphrase is confirmed correctly (like for the root password) instead
of bailing out after three failed attempts.

distrib/miniroot/install.sub

index 4386ec9..51f0828 100644 (file)
@@ -1,5 +1,5 @@
 #!/bin/ksh
-#      $OpenBSD: install.sub,v 1.1253 2023/08/10 17:09:34 kn Exp $
+#      $OpenBSD: install.sub,v 1.1254 2023/08/21 08:59:18 kn Exp $
 #
 # Copyright (c) 1997-2015 Todd Miller, Theo de Raadt, Ken Westerback
 # Copyright (c) 2015, Robert Peichaer <rpe@openbsd.org>
@@ -3075,7 +3075,7 @@ do_autoinstall() {
 }
 
 encrypt_root() {
-       local _chunk _tries=0
+       local _chunk
 
        [[ $MDBOOTSR == y ]] || return
 
@@ -3097,10 +3097,7 @@ encrypt_root() {
        md_prep_fdisk $_chunk
        echo 'RAID *' | disklabel -w -A -T- $_chunk
 
-       until bioctl -Cforce -cC -l${_chunk}a softraid0 >/dev/null; do
-               # Most likely botched passphrases, silently retry twice.
-               ((++_tries < 3)) || exit
-       done
+       bioctl -Cforce -cC -l${_chunk}a softraid0 >/dev/null
 
        # No volumes existed before asking, but we just created one.
        ROOTDISK=$(get_softraid_volumes)