From 14ef713434acb89e8c715eed7ba180a38fd604a9 Mon Sep 17 00:00:00 2001
From: Simo Sorce <simo@redhat.com>
Date: Fri, 23 Mar 2012 09:58:27 -0400
Subject: Avoid double free on error.

There is no need to expliciltly free here, as gp_rpc_process_call already
always frees the res structure. By doing this we cause double frees on errors.
---
 proxy/src/gp_rpc_accept_sec_context.c | 10 ----------
 1 file changed, 10 deletions(-)

(limited to 'proxy')

diff --git a/proxy/src/gp_rpc_accept_sec_context.c b/proxy/src/gp_rpc_accept_sec_context.c
index 62d2387..96c781b 100644
--- a/proxy/src/gp_rpc_accept_sec_context.c
+++ b/proxy/src/gp_rpc_accept_sec_context.c
@@ -129,16 +129,6 @@ done:
                                  ret_maj, ret_min, oid,
                                  &ascr->status);
 
-    if (ret_maj) {
-        if (ascr->context_handle) {
-            xdr_free((xdrproc_t)xdr_gssx_ctx, (char *)ascr->context_handle);
-            free(ascr->context_handle);
-        }
-        if (ascr->output_token) {
-            xdr_free((xdrproc_t)xdr_gssx_buffer, (char *)ascr->output_token);
-            free(ascr->output_token);
-        }
-    }
     gss_release_name(&ret_min, &src_name);
     gss_release_buffer(&ret_min, &obuf);
     gss_release_cred(&ret_min, &dch);
-- 
cgit