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
