The branch, master has been updated
       via  46989fb6e74 s3:passdb: Fix CIDs 1596750,1596751 uninitialized 
pointer read
      from  0ef50512489 libcli/security: use talloc_free at the end of sub-tests

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 46989fb6e74e2efed166c80f06174a10ac4daaf4
Author: Michael Adam <[email protected]>
Date:   Wed Aug 6 11:27:48 2025 +0200

    s3:passdb: Fix CIDs 1596750,1596751 uninitialized pointer read
    
    This fixes two coverity issues of type
    
    CID 1596750 - uninitialized pointer read
    CID 1596751 - uninitialized pointer read
    
     It fixes one occurrence in each of
    pdb_samba_dsdb_create_user and pdb_samba_dsdb_create_alias
    
    Signed-off-by: Michael Adam <[email protected]>
    Reviewed-by: Alexander Bokovoy <[email protected]>
    Reviewed-by: Rabinarayan Panigrahi <[email protected]>
    Reviewed-by: Shachar Sharon <[email protected]>
    
    Autobuild-User(master): Anoop C S <[email protected]>
    Autobuild-Date(master): Thu Aug  7 11:20:25 UTC 2025 on atb-devel-224

-----------------------------------------------------------------------

Summary of changes:
 source3/passdb/pdb_samba_dsdb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/passdb/pdb_samba_dsdb.c b/source3/passdb/pdb_samba_dsdb.c
index 0678f7516d1..96d9819316b 100644
--- a/source3/passdb/pdb_samba_dsdb.c
+++ b/source3/passdb/pdb_samba_dsdb.c
@@ -768,7 +768,7 @@ static NTSTATUS pdb_samba_dsdb_create_user(struct 
pdb_methods *m,
 {
        struct pdb_samba_dsdb_state *state = talloc_get_type_abort(
                m->private_data, struct pdb_samba_dsdb_state);
-       struct dom_sid *sid;
+       struct dom_sid *sid = NULL;
        struct ldb_dn *dn;
        NTSTATUS status;
        TALLOC_CTX *tmp_ctx = talloc_new(mem_ctx);
@@ -1582,7 +1582,7 @@ static NTSTATUS pdb_samba_dsdb_create_alias(struct 
pdb_methods *m,
        TALLOC_CTX *frame = talloc_stackframe();
        struct pdb_samba_dsdb_state *state = talloc_get_type_abort(
                m->private_data, struct pdb_samba_dsdb_state);
-       struct dom_sid *sid;
+       struct dom_sid *sid = NULL;
 
        struct ldb_dn *dn;
        NTSTATUS status;


-- 
Samba Shared Repository

Reply via email to