From 44dfbad623a8f1aeef917b63ed72d4a3caca95bf Mon Sep 17 00:00:00 2001
From: Douglas Teles <dga.teles@gmail.com>
Date: Wed, 24 Jun 2026 11:40:02 -0300
Subject: [PATCH] =?UTF-8?q?setrootpass:=20short-circuit=20via=20sha512cryp?=
 =?UTF-8?q?t=20sem=20novo=20artefato=20sens=C3=ADvel?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

A versão anterior do short-circuit gravava sha256(senha) em
/storage/.cache/.archr-rootpass.sha256 — fingerprint recuperável que
quebraria senhas curtas / dicionário por brute force trivial se o
arquivo vazasse. Revisão de segurança automatizada apontou.

Substituída a comparação por reproduzir o próprio sha512crypt já
armazenado em /storage/.cache/shadow: extrai o salt do campo CURRENT
(formato $6$<salt>$<hash>), chama cryptpw -m sha512 -S "$SALT"
"$ROOTPASS" e compara contra CURRENT. Nenhum novo arquivo é escrito;
o único hash de senha em disco continua sendo o sha512crypt salted
que já existia.

Live no R36S:
  - mesma senha: 0.147s (short-circuit)
  - senha nova: 1.233s (caminho completo)
  - nenhum arquivo extra em /storage/.cache/.archr-rootpass*

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 .../archr/sources/scripts/setrootpass         | 26 ++++++++++++-------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/projects/ArchR/packages/archr/sources/scripts/setrootpass b/projects/ArchR/packages/archr/sources/scripts/setrootpass
index 25dda003fa..b4d8c48ade 100755
--- a/projects/ArchR/packages/archr/sources/scripts/setrootpass
+++ b/projects/ArchR/packages/archr/sources/scripts/setrootpass
@@ -11,13 +11,22 @@ CURRENT=$(awk -F: '/^root/{print $2}' /storage/.cache/shadow)
 
 # Short-circuit no-op runs. 007-rootpw chama setrootpass a cada boot
 # com a senha já gravada — cryptpw sha512 + smbpasswd + syncthing
-# generate juntos custam ~6s mesmo quando nada vai mudar. Comparar a
-# senha contra o último valor escrito por nós e bail out se for igual.
-LAST_PASS_HASH="/storage/.cache/.archr-rootpass.sha256"
-if [ -n "${ROOTPASS}" ] && [ -f "${LAST_PASS_HASH}" ]; then
-  CURRENT_HASH=$(printf '%s' "${ROOTPASS}" | sha256sum | awk '{print $1}')
-  if [ "$(cat "${LAST_PASS_HASH}" 2>/dev/null)" = "${CURRENT_HASH}" ]; then
-    exit 0
+# generate juntos custam ~6s mesmo quando nada vai mudar.
+#
+# A comparação reusa o próprio hash sha512crypt já em /storage/.cache/
+# shadow: cryptpw -m sha512 -S "<sal>" "<senha>" reproduz o hash dado o
+# mesmo salt, então se o resultado bater com CURRENT a senha não mudou.
+# Isso evita escrever qualquer fingerprint adicional em disco — o
+# único hash de senha permanece o sha512crypt salted que já existia.
+if [ -n "${ROOTPASS}" ] && [ -n "${CURRENT}" ]; then
+  # sha512crypt: $6$<salt>$<hash>. O salt é o campo entre $6$ e o
+  # próximo $; awk extrai esse fragmento.
+  SALT=$(printf '%s\n' "${CURRENT}" | awk -F'$' '/^\$6\$/ {print $3}')
+  if [ -n "${SALT}" ]; then
+    RECOMPUTED=$(cryptpw -m sha512 -S "${SALT}" "${ROOTPASS}" 2>/dev/null)
+    if [ "${RECOMPUTED}" = "${CURRENT}" ]; then
+      exit 0
+    fi
   fi
 fi
 
@@ -36,9 +45,6 @@ syncthing generate --gui-user "root" --gui-password "${ROOTPASS}" >/dev/null 2>&
 # Save the password so we can display it in ES
 set_setting root.password "${ROOTPASS}"
 
-# Lembrar o hash da senha gravada para o short-circuit no próximo boot.
-printf '%s' "${ROOTPASS}" | sha256sum | awk '{print $1}' >"${LAST_PASS_HASH}" 2>/dev/null
-
 # Restart syncthing if it was enabled.
 SYNCSTATUS=$(get_setting syncthing.enabled)
 SYNCACTIVE=$(systemctl status syncthing | awk '/active/ {print $2}')