[SCM] Samba Shared Repository - branch v4-17-test updated

[Jule Anger]

The branch, v4-17-test has been updated
       via  d94cbb10b88 s3/utils: avoid erronous NO MEMORY detection
      from  9313ebba32b dsdb: Use samdb_system_container_dn() in pdb_samba_dsdb_*()

https://git.samba.org/?p=samba.git;a=shortlog;h=v4-17-test


- Log -----------------------------------------------------------------
commit d94cbb10b882d52782cbabc1328c751cea4cb86f
Author: Noel Power <noel.power at suse.com>
Date:   Fri Jun 2 14:27:55 2023 +0100

    s3/utils: avoid erronous NO MEMORY detection
    
    since 5cc3c1b5f6b0289f91c01b20989558badc28fd61 if we don't have
    a realm specified either on cmdline or in conf file we try to
    copy (talloc_strdup) a NULL variable which triggers a NO_MEMORY
    error when we check the result of the copy
    
    BUG: https://bugzilla.samba.org/show_bug.cgi?id=15384
    
    Signed-off-by: Noel Power <noel.power at suse.com>
    Reviewed-by: Stefan Metzmacher <metze at samba.org>
    
    Autobuild-User(master): Stefan Metzmacher <metze at samba.org>
    Autobuild-Date(master): Sun Jun  4 12:42:16 UTC 2023 on atb-devel-224
    
    (cherry picked from commit 22ab42c1007775abca0b578744d4c18a85cda627)
    
    Autobuild-User(v4-17-test): Jule Anger <janger at samba.org>
    Autobuild-Date(v4-17-test): Wed Aug  2 11:45:00 UTC 2023 on sn-devel-184

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

Summary of changes:
 source3/utils/net_ads.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/utils/net_ads.c b/source3/utils/net_ads.c
index dd9341f3637..70d05dc541a 100644
--- a/source3/utils/net_ads.c
+++ b/source3/utils/net_ads.c
@@ -713,10 +713,12 @@ retry:
 	} else if (ads->auth.realm == NULL) {
 		const char *c_realm = cli_credentials_get_realm(c->creds);
 
-		ads->auth.realm = talloc_strdup(ads, c_realm);
-		if (ads->auth.realm == NULL) {
-			TALLOC_FREE(ads);
-			return ADS_ERROR(LDAP_NO_MEMORY);
+		if (c_realm != NULL) {
+			ads->auth.realm = talloc_strdup(ads, c_realm);
+			if (ads->auth.realm == NULL) {
+				TALLOC_FREE(ads);
+				return ADS_ERROR(LDAP_NO_MEMORY);
+			}
 		}
 	}
 


-- 
Samba Shared Repository