Skip to content

Commit

Permalink
Fix t_s4u2proxy_krb5 cleanup
Browse files Browse the repository at this point in the history
Unlike most GSS test programs, t_s4u2proxy_krb5 uses a cleanup
handler, so we have to be careful to initialize everything we clean
up--particularly service2_name, which is initialized after a possible
goto.  Also, remember to release acceptor_name.
  • Loading branch information
greghudson committed Sep 24, 2012
1 parent b1d1418 commit 8f79585
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/tests/gssapi/t_s4u2proxy_krb5.c
Expand Up @@ -58,7 +58,8 @@ main(int argc, char *argv[])
gss_buffer_desc buf = GSS_C_EMPTY_BUFFER, token = GSS_C_EMPTY_BUFFER;
gss_OID mech;
gss_OID_set mechs;
gss_name_t acceptor_name, service1_name, service2_name, client_name;
gss_name_t acceptor_name = GSS_C_NO_NAME, client_name = GSS_C_NO_NAME;
gss_name_t service1_name = GSS_C_NO_NAME, service2_name = GSS_C_NO_NAME;
gss_cred_id_t service1_cred = GSS_C_NO_CREDENTIAL;
gss_cred_id_t deleg_cred = GSS_C_NO_CREDENTIAL;
gss_ctx_id_t initiator_context = GSS_C_NO_CONTEXT;
Expand Down Expand Up @@ -168,6 +169,7 @@ main(int argc, char *argv[])
(void)gss_release_buffer(&minor, &buf);

cleanup:
(void)gss_release_name(&minor, &acceptor_name);
(void)gss_release_name(&minor, &client_name);
(void)gss_release_name(&minor, &service1_name);
(void)gss_release_name(&minor, &service2_name);
Expand Down

0 comments on commit 8f79585

Please sign in to comment.