svn commit: r228514 - head/sys/net80211

[Bernhard Schmidt]

On Thu, Dec 15, 2011 at 01:52, Adrian Chadd <adrian at freebsd.org> wrote:
> Author: adrian
> Date: Thu Dec 15 00:52:30 2011
> New Revision: 228514
> URL: http://svn.freebsd.org/changeset/base/228514
>
> Log:
>  Modify the ACL code slightly to support a few nifty things:
>
>  * Call it before sending probe responses, so the ACL code has the
>    chance to reject sending them.
>
>  * Pass the whole frame to the ACL code now, rather than just the
>    destination MAC - that way the ACL module can look at the frame
>    contents to determine what the response should be.
>
>  This is part of some uncommitted work to support band steering.
>
>  Sponsored by: Hobnob, Inc.
>
> Modified:
>  head/sys/net80211/ieee80211_acl.c
>  head/sys/net80211/ieee80211_hostap.c
>  head/sys/net80211/ieee80211_mesh.c
>  head/sys/net80211/ieee80211_proto.h
>
> Modified: head/sys/net80211/ieee80211_acl.c
> ==============================================================================
> --- head/sys/net80211/ieee80211_acl.c   Wed Dec 14 23:57:47 2011        (r228513)
> +++ head/sys/net80211/ieee80211_acl.c   Thu Dec 15 00:52:30 2011        (r228514)
> @@ -152,7 +152,8 @@ _acl_free(struct aclstate *as, struct ac
>  }
>
>  static int
> -acl_check(struct ieee80211vap *vap, const uint8_t mac[IEEE80211_ADDR_LEN])
> +acl_check(struct ieee80211vap *vap, const struct ieee80211_frame *wh,
> +    const uint8_t mac[IEEE80211_ADDR_LEN])
>  {

Why didn't you remove the mac argument? It is assign from wh->i_addr2
anyways, seems rather too redundant to me.

-- 
Bernhard