(ANother) stall fixed, please update to HEAD

[PseudoCylon]

> ------------------------------
>
> Message: 5
> Date: Tue, 7 Aug 2012 12:34:52 -0400
> From: Kim Culhan <w8hdkim at gmail.com>
> Subject: Re: (ANother) stall fixed, please update to HEAD
> To: Adrian Chadd <adrian.chadd at gmail.com>
> Cc: freebsd-wireless at freebsd.org
> Message-ID:
>         <CAKZxVQVbKfW7WVuZjNFg5pGFu3Djzz=60FoSNTS83sz1zf72wQ at mail.gmail.com>
> Content-Type: text/plain; charset=ISO-8859-1
>
>
>> Yup. Would you be able to work with PseudoCylon and test his
>> ieee80211_iterate_nodes() patch? I'd like verification that it fixes
>> it for you before I tidy it up and commit it to -HEAD.
>
> iter.patch and iter2.patch cannot be applied in that order
> soo..  PseudoCylon could you please generate a diff
> against -HEAD from your present local source?
>

No it won't. Only one of them need to apply. I guess I didn't explain well.

iter.path only print outs debug message when the array overflowed
(most unlikely it will). At this moment, this is what we need.

iter2.patch does iter.patch + revert changes + abort iterating just
for piece of mind. Probably this is unnecessary. The code need to be
patched in the way the array won't overflow if it ever happens. (I
leave it to committers what to commit.)

The attached patch can be applied over iter.patch

Sorry for the confusion.


AK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: addon.patch
Type: application/octet-stream
Size: 2058 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-wireless/attachments/20120809/ddda632f/addon.obj