diff options
| author | Jeremy Allison <jeremy@jeremy-desktop.(none)> | 2008-12-23 12:10:47 -0800 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2009-01-02 12:44:24 +0100 |
| commit | bfaec4b40353d91a92ffd0d6939b1a5d4df72a0e (patch) | |
| tree | 622130ce0a364fe329399b6de71a36c7d78b4529 /source/libads | |
| parent | 6ddb477052eb50ad1154468c71e6fe804b3127e0 (diff) | |
| download | samba-bfaec4b40353d91a92ffd0d6939b1a5d4df72a0e.tar.gz samba-bfaec4b40353d91a92ffd0d6939b1a5d4df72a0e.tar.xz samba-bfaec4b40353d91a92ffd0d6939b1a5d4df72a0e.zip | |
Fix more asprintf warnings and some error path errors.
Jeremy.
(cherry picked from commit 44ab658b6d80c66f20d80aede7fc7cd2b9a941b1)
Diffstat (limited to 'source/libads')
| -rw-r--r-- | source/libads/util.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/source/libads/util.c b/source/libads/util.c index d23c36f3269..1fc057996d9 100644 --- a/source/libads/util.c +++ b/source/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); |