On Tue, Jul 14, 2015 at 09:13:49PM +0200, Joerg Mayer wrote: > 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)
With current git head all the messages are gone. Many thanks! Jörg -- Joerg Mayer <jma...@loplof.de> 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 <wireshark-dev@wireshark.org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe