]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s3:registry: Initialize struct security_ace ace[]
authorPavel Filipenský <pfilipensky@samba.org>
Wed, 3 Jul 2024 15:34:47 +0000 (17:34 +0200)
committerVolker Lendecke <vl@samba.org>
Fri, 5 Jul 2024 07:45:32 +0000 (07:45 +0000)
Error: UNINIT (CWE-457):
samba-4.20.0rc2/source3/registry/reg_dispatcher.c:43: var_decl: Declaring variable "ace" without initializer.
samba-4.20.0rc2/source3/registry/reg_dispatcher.c:66: uninit_use_in_call: Using uninitialized value "*ace". Field "ace->object" is uninitialized when calling "make_sec_acl".
  64|           /* create the security descriptor */
  65|
  66|->         theacl = make_sec_acl(ctx, NT4_ACL_REVISION, i, ace);
  67|           if (theacl == NULL) {
  68|                   return WERR_NOT_ENOUGH_MEMORY;

Signed-off-by: Pavel Filipenský <pfilipensky@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
source3/registry/reg_dispatcher.c

index ab3fb24dc2ff4efc4955cf88bed979e037a8fa7e..097b6d4d30fc45c65769dd58f23bc9e0982a0685 100644 (file)
@@ -40,7 +40,7 @@ static const struct generic_mapping reg_generic_map =
 
 static WERROR construct_registry_sd(TALLOC_CTX *ctx, struct security_descriptor **psd)
 {
-       struct security_ace ace[3];
+       struct security_ace ace[3] = {};
        size_t i = 0;
        struct security_descriptor *sd;
        struct security_acl *theacl;