From: Tobias Brunner <tobias@strongswan.org>
Date: Wed, 21 Nov 2012 10:55:55 +0000 (+0100)
Subject: Fixed two bugs in logging MSK during EAP-SIM/AKA
X-Git-Tag: 5.0.2dr4~178
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=35ba45916cba3bc979f36d745a18dd0ea3e744b9;p=thirdparty%2Fstrongswan.git

Fixed two bugs in logging MSK during EAP-SIM/AKA

msk is already a pointer to a chunk_t and it was actually not set yet.
---

diff --git a/src/libsimaka/simaka_crypto.c b/src/libsimaka/simaka_crypto.c
index 8b47198dfb..e60c02a1a6 100644
--- a/src/libsimaka/simaka_crypto.c
+++ b/src/libsimaka/simaka_crypto.c
@@ -150,7 +150,6 @@ METHOD(simaka_crypto_t, derive_keys_full, bool,
 
 	k_encr = chunk_create(str.ptr, KENCR_LEN);
 	k_auth = chunk_create(str.ptr + KENCR_LEN, KAUTH_LEN);
-	DBG3(DBG_LIB, "K_encr %B\nK_auth %B\nMSK %B", &k_encr, &k_auth, &msk);
 
 	if (!this->signer->set_key(this->signer, k_auth) ||
 		!this->crypter->set_key(this->crypter, k_encr))
@@ -160,6 +159,7 @@ METHOD(simaka_crypto_t, derive_keys_full, bool,
 	}
 
 	*msk = chunk_clone(chunk_create(str.ptr + KENCR_LEN + KAUTH_LEN, MSK_LEN));
+	DBG3(DBG_LIB, "K_encr %B\nK_auth %B\nMSK %B", &k_encr, &k_auth, msk);
 
 	call_hook(this, k_encr, k_auth);