summaryrefslogtreecommitdiffstats
path: root/src/lib
diff options
context:
space:
mode:
authorSam Hartman <hartmans@mit.edu>2010-02-09 19:15:12 +0000
committerSam Hartman <hartmans@mit.edu>2010-02-09 19:15:12 +0000
commit19f23a3e34fe581e986d579b0bd87abfce664365 (patch)
treef79e5d7c9b36c571b4a26195f35f4c1c2b71eba0 /src/lib
parent7a04b369aa71b0796f057cf5da57070a9c37c9c8 (diff)
downloadkrb5-19f23a3e34fe581e986d579b0bd87abfce664365.tar.gz
krb5-19f23a3e34fe581e986d579b0bd87abfce664365.tar.xz
krb5-19f23a3e34fe581e986d579b0bd87abfce664365.zip
krb5int_fast_free_state segfaults if state is null
krb5int_fast_free_state fails if state is null. INstead it should simply return Reorganization of the get_init_creds logic has created situations where the init_creds loop can fail between the time when the context is initialized and the fast state is initialized. ticket: 6657 target_version: 1.8 tags: pullup git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@23713 dc483132-0cff-0310-8789-dd5450dbe970
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/krb5/krb/fast.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/lib/krb5/krb/fast.c b/src/lib/krb5/krb/fast.c
index db95adca71..6aa71282ab 100644
--- a/src/lib/krb5/krb/fast.c
+++ b/src/lib/krb5/krb/fast.c
@@ -533,6 +533,8 @@ krb5int_fast_make_state( krb5_context context, struct krb5int_fast_request_state
void
krb5int_fast_free_state( krb5_context context, struct krb5int_fast_request_state *state)
{
+ if (state == NULL)
+ return;
/*We are responsible for none of the store in the fast_outer_req*/
krb5_free_keyblock(context, state->armor_key);
krb5_free_fast_armor(context, state->armor);