commit 583ecbe728ec53e4b8cc9a1ecb7bc67a847bb008 Author: tlyu tlyu@dc483132-0cff-0310-8789-dd5450dbe970 Date: Fri Feb 12 20:28:43 2010 +0000

ticket: 6657 version_fixed: 1.8 status: resolved

pull up r23713 from trunk

------------------------------------------------------------------------ r23713 | hartmans | 2010-02-09 14:15:12 -0500 (Tue, 09 Feb 2010) | 10 lines

subject: krb5int_fast_free_state segfaults if state is null ticket: 6657 target_version: 1.8 tags: pullup

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.

git-svn-id: svn://anonsvn.mit.edu/krb5/branches/krb5-1-8@23719 dc483132-0cff-0310-8789-dd5450dbe970

src/lib/krb5/krb/fast.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) --- diff --git a/src/lib/krb5/krb/fast.c b/src/lib/krb5/krb/fast.c index 2ebbdcf..1ff2d67 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);