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@suse.com>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Sun Jun  4 12:42:16 UTC 2023 on atb-devel-224

(cherry picked from commit 22ab42c1007775abca0b578744d4c18a85cda627)

diff --git a/source3/utils/net_ads.c b/source3/utils/net_ads.c
index 9ec884394eb049948c958923753b96dd5b110bb4..ff56d7f76e2f3f222e6107caec3df4e84dd5cfee 100644 (file)
--- 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);
+                       }
                }
        }