From 526e875cec15761099438e17df3f56bc2bd5b761 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Wed, 8 Aug 2012 15:35:28 -0700 Subject: Check error returns from strupper_m() (in all reasonable places). --- source3/libnet/libnet_join.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'source3/libnet') diff --git a/source3/libnet/libnet_join.c b/source3/libnet/libnet_join.c index 399c13a86a..5806f7bc75 100644 --- a/source3/libnet/libnet_join.c +++ b/source3/libnet/libnet_join.c @@ -136,7 +136,10 @@ static ADS_STATUS libnet_connect_ads(const char *dns_domain_name, *cp++ = '\0'; SAFE_FREE(my_ads->auth.realm); my_ads->auth.realm = smb_xstrdup(cp); - strupper_m(my_ads->auth.realm); + if (!strupper_m(my_ads->auth.realm)) { + ads_destroy(&my_ads); + return ADS_ERROR_LDAP(LDAP_NO_MEMORY); + } } } @@ -381,7 +384,9 @@ static ADS_STATUS libnet_join_set_machine_spn(TALLOC_CTX *mem_ctx, if (!spn) { return ADS_ERROR_LDAP(LDAP_NO_MEMORY); } - strupper_m(spn); + if (!strupper_m(spn)) { + return ADS_ERROR_LDAP(LDAP_NO_MEMORY); + } spn_array[0] = spn; if (!name_to_fqdn(my_fqdn, r->in.machine_name) -- cgit