On Tue, Jul 14, 2015 at 11:52:18AM -0700, Guy Harris wrote:
> Line 1558 of epan/crypt/airpdcap.c is
> 
>       if (ctx->sa[ctx->first_free_index].used) { 
> 
> in AirPDcapStoreSa().  It was assuming that ctx->first_free_index would be 
> within the bounds of the array, which isn't guaranteed (what if there *are* 
> no free indices?); I've added a bounds check in 
> 4f1b8d74338ca2a6ded8498e9d87cbc3294454c0.

This was on Linux (which has AIRPCAP disabled) and with only 2 entries total
(1x wpa, 1x wpa2)

Thanks!
   Jörg
-- 
Joerg Mayer                                           <[email protected]>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe

Reply via email to