diff options
author | Greg Hudson <ghudson@mit.edu> | 2012-08-11 00:16:25 -0400 |
---|---|---|
committer | Greg Hudson <ghudson@mit.edu> | 2012-08-11 00:28:33 -0400 |
commit | 1870feda057e24826f505eb5f3151aad3041ab1e (patch) | |
tree | b1b0368ccaf818979d541a56e711572c66ac5c25 /src/lib/rpc/auth_gssapi.c | |
parent | b0edf38aafe3a365821cae4874e4608f95e28896 (diff) | |
download | krb5-1870feda057e24826f505eb5f3151aad3041ab1e.tar.gz krb5-1870feda057e24826f505eb5f3151aad3041ab1e.tar.xz krb5-1870feda057e24826f505eb5f3151aad3041ab1e.zip |
Stop misusing gss_release_buffer in libgssrpc
Use free() instead of gss_release_buffer() when freeing buffers in
libgssrpc which weren't constructed by GSSAPI. This mixing is
harmless in normal configurations (since libgssrpc is only used on
Unix), but fails with DEBUG_GSSALLOC.
Diffstat (limited to 'src/lib/rpc/auth_gssapi.c')
-rw-r--r-- | src/lib/rpc/auth_gssapi.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/lib/rpc/auth_gssapi.c b/src/lib/rpc/auth_gssapi.c index e7a1f8b00b..64a6b5b791 100644 --- a/src/lib/rpc/auth_gssapi.c +++ b/src/lib/rpc/auth_gssapi.c @@ -743,9 +743,7 @@ skip_call: gssstat, minor_stat)); } - if (AUTH_PRIVATE(auth)->client_handle.length != 0) - gss_release_buffer(&minor_stat, - &AUTH_PRIVATE(auth)->client_handle); + free(AUTH_PRIVATE(auth)->client_handle.value); #if 0 PRINTF(("gssapi_destroy: calling GSSAPI_EXIT\n")); |