Fixed two bugs in logging MSK during EAP-SIM/AKA
authorTobias Brunner <tobias@strongswan.org>
Wed, 21 Nov 2012 10:55:55 +0000 (11:55 +0100)
committerTobias Brunner <tobias@strongswan.org>
Wed, 21 Nov 2012 10:55:55 +0000 (11:55 +0100)
msk is already a pointer to a chunk_t and it was actually not set yet.

src/libsimaka/simaka_crypto.c

index 8b47198..e60c02a 100644 (file)
@@ -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);