Although it can't actually happen, make it more explicit that we won't
authorGreg Hudson <ghudson@mit.edu>
Fri, 11 Mar 2011 17:53:18 +0000 (17:53 +0000)
committerGreg Hudson <ghudson@mit.edu>
Fri, 11 Mar 2011 17:53:18 +0000 (17:53 +0000)
dereference a null mech in the cleanup handler of the mechglue's
gss_accept_sec_context.

ticket: 6813

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@24701 dc483132-0cff-0310-8789-dd5450dbe970

src/lib/gssapi/mechglue/g_accept_sec_context.c

index cf21c151343c966e01fe567aa5dee7624f8fb877..8714f3fb7b325285b229c963825d1b15fc7d8351 100644 (file)
@@ -353,7 +353,8 @@ error_out:
                free(union_ctx_id->mech_type->elements);
            free(union_ctx_id->mech_type);
        }
-       if (union_ctx_id->internal_ctx_id && mech->gss_delete_sec_context) {
+       if (union_ctx_id->internal_ctx_id && mech &&
+           mech->gss_delete_sec_context) {
            mech->gss_delete_sec_context(&temp_minor_status,
                                         &union_ctx_id->internal_ctx_id,
                                         GSS_C_NO_BUFFER);