diff options
| author | Jeremy Allison <jeremy@jeremy-desktop.(none)> | 2008-12-23 12:11:12 -0800 |
|---|---|---|
| committer | Jeremy Allison <jeremy@jeremy-desktop.(none)> | 2008-12-23 12:11:12 -0800 |
| commit | d0eeb9aa8db42ed1fc18024614560820354e6dfa (patch) | |
| tree | 4b981ba4aa23fe1a9d53139a135018cbbd5871b5 /source3/libads/util.c | |
| parent | 1966a947d3f8d432d3af5a9c0004e27561ae7f65 (diff) | |
| download | samba-d0eeb9aa8db42ed1fc18024614560820354e6dfa.tar.gz samba-d0eeb9aa8db42ed1fc18024614560820354e6dfa.tar.bz2 samba-d0eeb9aa8db42ed1fc18024614560820354e6dfa.zip | |
Fix more asprintf warnings and some error path errors.
Jeremy.
Diffstat (limited to 'source3/libads/util.c')
| -rw-r--r-- | source3/libads/util.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/source3/libads/util.c b/source3/libads/util.c index 9866a15285..2c7ccfebd6 100644 --- a/source3/libads/util.c +++ b/source3/libads/util.c @@ -71,7 +71,11 @@ ADS_STATUS ads_guess_service_principal(ADS_STRUCT *ads, strlower_m(server); strupper_m(server_realm); - asprintf(&princ, "ldap/%s@%s", server, server_realm); + if (asprintf(&princ, "ldap/%s@%s", server, server_realm) == -1) { + SAFE_FREE(server); + SAFE_FREE(server_realm); + return ADS_ERROR(LDAP_NO_MEMORY); + } SAFE_FREE(server); SAFE_FREE(server_realm); @@ -93,7 +97,11 @@ ADS_STATUS ads_guess_service_principal(ADS_STRUCT *ads, strlower_m(server); strupper_m(server_realm); - asprintf(&princ, "ldap/%s@%s", server, server_realm); + if (asprintf(&princ, "ldap/%s@%s", server, server_realm) == -1) { + SAFE_FREE(server); + SAFE_FREE(server_realm); + return ADS_ERROR(LDAP_NO_MEMORY); + } SAFE_FREE(server); SAFE_FREE(server_realm); |