aboutsummaryrefslogtreecommitdiff
path: root/src/lib/gssapi
diff options
context:
space:
mode:
authorGreg Hudson <ghudson@mit.edu>2011-03-11 17:53:18 +0000
committerGreg Hudson <ghudson@mit.edu>2011-03-11 17:53:18 +0000
commit8647bcedc38882da7ea1c22c973faedc0b571ceb (patch)
treef753296cde4d2b1b3803a74aed942751fa89caee /src/lib/gssapi
parent16007fbb9361745cef2a9c322e0b7e1ec2b9d77a (diff)
downloadkrb5-8647bcedc38882da7ea1c22c973faedc0b571ceb.zip
krb5-8647bcedc38882da7ea1c22c973faedc0b571ceb.tar.gz
krb5-8647bcedc38882da7ea1c22c973faedc0b571ceb.tar.bz2
Although it can't actually happen, make it more explicit that we won't
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
Diffstat (limited to 'src/lib/gssapi')
-rw-r--r--src/lib/gssapi/mechglue/g_accept_sec_context.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/lib/gssapi/mechglue/g_accept_sec_context.c b/src/lib/gssapi/mechglue/g_accept_sec_context.c
index cf21c15..8714f3f 100644
--- a/src/lib/gssapi/mechglue/g_accept_sec_context.c
+++ b/src/lib/gssapi/mechglue/g_accept_sec_context.c
@@ -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);