commit 1b2205b304af61d6a417b2770b181088c14333a8
author Zesstra <zesstra@zesstra.de> Sat Jun 28 15:56:05 2025 +0200
committer Zesstra <zesstra@zesstra.de> Sat Jun 28 16:08:12 2025 +0200
tree b66c4ced8f5fd055b72e50bcaf31460b596d6f37
parent 68a28c25d2f6d701fbdabb778cef31e4b6f1003c

Zugriffscheck beim Setzen von SECURED gefixt

Auch das Setzen von SECURED wurde in der Pruefung wie das
Loeschen behandelt und demnach zu oft abgelehnt (z.b. bei
ARCH_SECURITY).

Change-Id: I10f93c397bc50f3c7243102dcd250bd9d4245e59

std/thing/properties.c

diff --git a/std/thing/properties.c b/std/thing/properties.c
index ee48a52..4f5e18b 100644
--- a/std/thing/properties.c
+++ b/std/thing/properties.c
@@ -126,12 +126,14 @@
   // koennen es ROOT, EM+ und ME ueber den Check oben.
 
   // Soll SECURED geloescht werden und ist das erlaubt?
-  if ((Type==F_MODE||Type==F_MODE_AD) && (Value & SECURED)
+  if ( (Type==F_MODE||Type==F_MODE_AD) && (Value & SECURED)
+      && (prop[F_MODE][name] & SECURED)
       && !prop_check_privilege(drop_priv, mode, SECURED_DELETE))
     return -2;
 
   // Soll SECURED gesetzt werden und ist das erlaubt?
-  if ((Type==F_MODE||Type==F_MODE_AS) && (Value&SECURED)
+  if ( (Type==F_MODE||Type==F_MODE_AS) && (Value&SECURED)
+      && !(prop[F_MODE][name] & SECURED)
       && !prop_check_privilege(drop_priv, mode, SECURED_SET) )
     return -3;