Re: [Ieee80211-devel] [PATCH] Assoc header fix

[Ivo v. D. <iv...@gm...>]

On Thursday 28 July 2005 17:10, Pedro Ramalhais wrote:
> Ivo van Doorn wrote:
> > Hi,
> >
> > This patch will add the ieee80211_hdr_3addr to the
> > ieee80211_assoc_request_frame structure.
> >
> > Patch is against ieee80211 1.0.3.
> >
> > IvD
> >
> >
> > diff -U 3 -H -w -E -d -r -N -- ieee80211-1.0.3/net/ieee80211.h
> > ieee80211-1.0.3-assoc/net/ieee80211.h
> > --- ieee80211-1.0.3/net/ieee80211.h     2005-07-15 01:53:35.000000000
> > +0200 +++ ieee80211-1.0.3-assoc/net/ieee80211.h       2005-07-28
> > 16:58:30.000000000 +0200
> > @@ -566,6 +566,7 @@
> >  } __attribute__ ((packed));
> >
> >  struct ieee80211_assoc_request_frame {
> > +       struct ieee80211_hdr_3addr header;
> >         u16 capability;
> >         u16 listen_interval;
> >         u8 current_ap[ETH_ALEN];
>
> I don't get it. What's that good for?
> Thanks!

It will make more sense since the structures
ieee80211_authentication
ieee80211_probe_response
ieee80211_assoc_response_frame
also contain this ieee80211_hdr_3addr  header.
I cannot see a reason why it should not be in ieee80211_assoc_request_frame.

IvD