On Thu, Aug 03, 2017 at 03:20:13PM -0300, R0me0 *** wrote:
> Yes, all on raspberry
> 
> Stefan, I can setup  VM and test amd64. Current or 6.1-stable?
> 
> To ensure I didn't a mess follow the diff:
> 
> diff -u ieee80211_proto-orig.c /usr/src/sys/net80211/ieee80211_proto.c

This diff is incorrect, it deletes way too many lines.

> --- ieee80211_proto-orig.c      Thu Aug  3 13:38:26 2017
> +++ /usr/src/sys/net80211/ieee80211_proto.c     Thu Aug  3 13:10:45 2017
> @@ -1023,10 +1023,6 @@
>                case IEEE80211_S_INIT:
>                case IEEE80211_S_SCAN:
>                case IEEE80211_S_ASSOC:
> -                       panic("invalid transition %s -> %s",
> -                           ieee80211_state_name[ostate],
> -                           ieee80211_state_name[nstate]);
> -                       break;
>                case IEEE80211_S_AUTH:
>                        IEEE80211_SEND_MGMT(ic, ni,
>                            IEEE80211_FC0_SUBTYPE_ASSOC_REQ, 0);

Here is my diff again (move just the "IEEE80211_S_ASSOC" line down).

Index: ieee80211_proto.c
===================================================================
RCS file: /cvs/src/sys/net80211/ieee80211_proto.c,v
retrieving revision 1.78
diff -u -p -r1.78 ieee80211_proto.c
--- ieee80211_proto.c   22 Jul 2017 16:54:46 -0000      1.78
+++ ieee80211_proto.c   28 Jul 2017 09:05:55 -0000
@@ -1022,11 +1022,11 @@ justcleanup:
                switch (ostate) {
                case IEEE80211_S_INIT:
                case IEEE80211_S_SCAN:
-               case IEEE80211_S_ASSOC:
                        panic("invalid transition %s -> %s",
                            ieee80211_state_name[ostate],
                            ieee80211_state_name[nstate]);
                        break;
+               case IEEE80211_S_ASSOC:
                case IEEE80211_S_AUTH:
                        IEEE80211_SEND_MGMT(ic, ni,
                            IEEE80211_FC0_SUBTYPE_ASSOC_REQ, 0);

Reply via email to