[SCM] Samba Shared Repository - branch master updated

[Ralph Böhme]

The branch, master has been updated
       via  3be1e68 winbind: Fix CID 1427626 Uninitialized scalar variable
       via  4e5c942 pdb: Fix CID 1427620 Resource leak
       via  92131d0 winbind: Fix CID 1427626 Uninitialized scalar variable
       via  e4f62d4 pdb: Fix CID 1427624 Resource leak
      from  0edce86 vfs_glusterfs: Add fallocate support for vfs_glusterfs

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


- Log -----------------------------------------------------------------
commit 3be1e68ce69f7ab8ac2cac97920c0e7f65b5ed6f
Author: Volker Lendecke <vl at samba.org>
Date:   Wed Jan 17 14:45:49 2018 +0100

    winbind: Fix CID 1427626 Uninitialized scalar variable
    
    Likely a false positive, but Coverity can't follow all the paths leading
    to line 1598.
    
    Signed-off-by: Volker Lendecke <vl at samba.org>
    Reviewed-by: Ralph Boehme <slow at samba.org>
    
    Autobuild-User(master): Ralph Böhme <slow at samba.org>
    Autobuild-Date(master): Wed Jan 17 23:58:34 CET 2018 on sn-devel-144

commit 4e5c9427085f001941adaf761b18740a2e169240
Author: Volker Lendecke <vl at samba.org>
Date:   Wed Jan 17 14:38:41 2018 +0100

    pdb: Fix CID 1427620 Resource leak
    
    It's not exactly a resource leak (we only really realloc if we shrink
    dramatically), but assigning the result from tdb_realloc looks nicer.
    
    Signed-off-by: Volker Lendecke <vl at samba.org>
    Reviewed-by: Ralph Boehme <slow at samba.org>

commit 92131d08987ac7c2fb219bf2e8300f3bc7b702f9
Author: Volker Lendecke <vl at samba.org>
Date:   Wed Jan 17 14:42:31 2018 +0100

    winbind: Fix CID 1427626 Uninitialized scalar variable
    
    Likely a false positive, but Coverity can't follow all the paths leading
    to line 2030
    
    Signed-off-by: Volker Lendecke <vl at samba.org>
    Reviewed-by: Ralph Boehme <slow at samba.org>

commit e4f62d4e4b91395d46c99c2a5313b0536793cca7
Author: Volker Lendecke <vl at samba.org>
Date:   Wed Jan 17 14:38:41 2018 +0100

    pdb: Fix CID 1427624 Resource leak
    
    It's not exactly a resource leak (we only really realloc if we shrink
    dramatically), but assigning the result from tdb_realloc looks nicer.
    
    Signed-off-by: Volker Lendecke <vl at samba.org>
    Reviewed-by: Ralph Boehme <slow at samba.org>

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

Summary of changes:
 source3/passdb/pdb_samba_dsdb.c | 5 +++--
 source3/winbindd/winbindd_pam.c | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/passdb/pdb_samba_dsdb.c b/source3/passdb/pdb_samba_dsdb.c
index 16a7a85..4e55a15 100644
--- a/source3/passdb/pdb_samba_dsdb.c
+++ b/source3/passdb/pdb_samba_dsdb.c
@@ -2959,7 +2959,7 @@ static NTSTATUS pdb_samba_dsdb_enum_trusteddoms(struct pdb_methods *m,
 		domains[di++] = d;
 	}
 
-	talloc_realloc(domains, domains, struct trustdom_info *, di);
+	domains = talloc_realloc(domains, domains, struct trustdom_info *, di);
 	*_domains = talloc_move(mem_ctx, &domains);
 	*_num_domains = di;
 	TALLOC_FREE(tmp_ctx);
@@ -3741,7 +3741,8 @@ static NTSTATUS pdb_samba_dsdb_enum_trusted_domains(struct pdb_methods *m,
 		domains[di++] = d;
 	}
 
-	talloc_realloc(domains, domains, struct pdb_trusted_domain *, di);
+	domains = talloc_realloc(domains, domains, struct pdb_trusted_domain *,
+				 di);
 	*_domains = talloc_move(mem_ctx, &domains);
 	*_num_domains = di;
 	TALLOC_FREE(tmp_ctx);
diff --git a/source3/winbindd/winbindd_pam.c b/source3/winbindd/winbindd_pam.c
index 9a61cd3..97425e8 100644
--- a/source3/winbindd/winbindd_pam.c
+++ b/source3/winbindd/winbindd_pam.c
@@ -1428,7 +1428,7 @@ static NTSTATUS winbind_samlogon_retry_loop(struct winbindd_domain *domain,
 	int netr_attempts = 0;
 	bool retry = false;
 	NTSTATUS result;
-	uint16_t validation_level;
+	uint16_t validation_level = UINT16_MAX;
 	union netr_Validation *validation = NULL;
 
 	do {
@@ -1823,7 +1823,7 @@ enum winbindd_result winbindd_dual_pam_auth(struct winbindd_domain *domain,
 	fstring name_domain, name_user;
 	char *mapped_user;
 	fstring domain_user;
-	uint16_t validation_level;
+	uint16_t validation_level = UINT16_MAX;
 	union netr_Validation *validation = NULL;
 	NTSTATUS name_map_status = NT_STATUS_UNSUCCESSFUL;
 


-- 
Samba Shared Repository