Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 21 Oct 2010 19:28:52 +0000 (UTC)
From:      Bernhard Schmidt <bschmidt@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r214160 - head/sys/dev/iwi
Message-ID:  <201010211928.o9LJSqgS001275@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bschmidt
Date: Thu Oct 21 19:28:52 2010
New Revision: 214160
URL: http://svn.freebsd.org/changeset/base/214160

Log:
  Instead of calling return when reaching the end of the assoc notification
  break the loop instead. We want to run the code after the while loop
  to set an associd and capinfo. If we don't do this net80211 will drop
  frames because it assumes the node has not yet been associated.
  
  MFC after:	1 week

Modified:
  head/sys/dev/iwi/if_iwi.c

Modified: head/sys/dev/iwi/if_iwi.c
==============================================================================
--- head/sys/dev/iwi/if_iwi.c	Thu Oct 21 19:27:27 2010	(r214159)
+++ head/sys/dev/iwi/if_iwi.c	Thu Oct 21 19:28:52 2010	(r214160)
@@ -1356,7 +1356,7 @@ iwi_checkforqos(struct ieee80211vap *vap
 
 	wme = NULL;
 	while (frm < efrm) {
-		IEEE80211_VERIFY_LENGTH(efrm - frm, frm[1], return);
+		IEEE80211_VERIFY_LENGTH(efrm - frm, frm[1], break);
 		switch (*frm) {
 		case IEEE80211_ELEMID_VENDOR:
 			if (iswmeoui(frm))



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201010211928.o9LJSqgS001275>