Ticket #579 (closed enhancement: fixed)

Opened 6 years ago

Last modified 6 years ago

if no kernel AES crypto support, wlan_ccmp loads but fails to attach w/o listing reason

Reported by: nick@kreucher.net Assigned to: kelmo
Priority: minor Milestone: version 0.9.1
Component: madwifi: 802.11 stack Version:
Keywords: Cc:
Patch is attached: 1 Pending:

Description

wpa_supplicant reports:

WPA: Installing PTK to the driver. wpa_driver_madwifi_set_key: alg=CCMP key_idx=0 set_tx=1 seq_len=6 key_len=16 ioctl[IEEE80211_IOCTL_SETKEY]: No such device or address WPA: Failed to set PTK to the driver.

Setting 80211debug +crypto +wpa revels only:

ath0: ieee80211_crypto_newkey: unable to attach cipher AES-CCM

But wlan_ccmp module is loaded.

Did some digging... and then added some more verbose debug messages to net80211/ieee80211_crypto_ccmp.c. From this I discovered wlan_ccmp was not able to load AES crypto support; I simply didn't have it compiled in my kernel - blah!

Since it took WAY to long to discover this, it might be a good idea to keep debug info similar to the following in net80211/ieee80211_crypto_ccmp.c (aka I did not realize wlan_ccmp would need to load AES crypto support outside of the madwifi modules):

--- ieee80211_crypto_ccmp.c     2006-04-28 11:26:29.000000000 -0700
+++ ieee80211_crypto_ccmp.c-nick        2006-04-28 11:26:19.000000000 -0700
@@ -107,6 +110,9 @@
        ctx->cc_ic = vap->iv_ic;
        ctx->cc_tfm = crypto_alloc_tfm("aes", 0);
        if (ctx->cc_tfm == NULL) {
+               IEEE80211_DPRINTF(vap, IEEE80211_MSG_CRYPTO,
+                       "%s: unable to load kernel AES crypto support\n",
+                       __func__);
                FREE(ctx, M_DEVBUF);
                _MOD_DEC_USE(THIS_MODULE);
                return NULL;

Attachments

wlan_ccmp-aes_debug.patch (458 bytes) - added by nick@kreucher.net on 05/02/06 17:27:21.
Signed-off-by: Nicholas J Kreucher <nick@kreucher.net>

Change History

04/29/06 05:32:27 changed by kelmo

Nick, thanks for the patch and information about your experiences. Can I bother you to do two things:-

  1. Attach the patch as a file to this ticket, rather than putting it inline with the contents of the ticket
  2. Sign-off the patch.

Perhaps we should also do a Makefile sanity check for this? Comments?

Thanks, Kel.

05/02/06 10:53:11 changed by mrenzmann

  • milestone set to version 0.9.x - progressive release candidate phase.

05/02/06 17:27:21 changed by nick@kreucher.net

  • attachment wlan_ccmp-aes_debug.patch added.

Signed-off-by: Nicholas J Kreucher <nick@kreucher.net>

05/23/06 12:15:48 changed by kelmo

  • status changed from new to assigned.
  • owner set to kelmo.

Better take assignment of this ticket so it does not get lost.

06/02/06 15:01:58 changed by kelmo

  • status changed from assigned to closed.
  • resolution set to fixed.

Thanks Nicholas, Patch applied to r1624.

06/19/06 09:19:47 changed by mrenzmann

  • milestone changed from version 0.9.x - progressive release candidate phase to version 0.9.1.

Add/Change #579 (if no kernel AES crypto support, wlan_ccmp loads but fails to attach w/o listing reason)