diff options
author | Volker Lendecke <vl@samba.org> | 2009-02-13 09:53:56 -0800 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2009-02-16 09:57:04 +0100 |
commit | c02bde0dbe5e5a498dfd922ecb349ba912821d1d (patch) | |
tree | 53611911daeda491f9aca950c14c7eb809fd7ffd /source | |
parent | 249738a5c45a970477358761983e20f2d1a8e794 (diff) | |
download | samba-c02bde0dbe5e5a498dfd922ecb349ba912821d1d.tar.gz |
Fix Coverity ID 744
This was marked as a resource leak. This change makes the code a bit clearer
that we always free error_string.
(cherry picked from commit fa0ac60d985995a6a07aa05b114683f4cbd731c8)
Diffstat (limited to 'source')
-rw-r--r-- | source/utils/ntlm_auth.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/utils/ntlm_auth.c b/source/utils/ntlm_auth.c index 2c62bd5c88f..8a9ac42329a 100644 --- a/source/utils/ntlm_auth.c +++ b/source/utils/ntlm_auth.c @@ -1816,7 +1816,6 @@ static void manage_ntlm_server_1_request(struct ntlm_auth_state *state, x_fprintf(x_stdout, "Authenticated: No\n"); x_fprintf(x_stdout, "Authentication-Error: %s\n.\n", error_string); - SAFE_FREE(error_string); } else { static char zeros[16]; char *hex_lm_key; @@ -1844,6 +1843,7 @@ static void manage_ntlm_server_1_request(struct ntlm_auth_state *state, TALLOC_FREE(hex_user_session_key); } } + SAFE_FREE(error_string); } /* clear out the state */ challenge = data_blob_null; |