Gerald Combs wrote:
> The 802.11 dissector has the following code:
> 
>   wlan_hdr *volatile whdr;
>   static wlan_hdr whdrs[4];
> 
>   whdr= &whdrs[0];
> 
> It looks like whdrs[1] through whdrs[3] are never used.  Should the code
> be modified to rotate through each member of whdrs, similar to the
> fmtbuf arrays in strutil.c?

Yes, although this would probably make a difference only if you have 
802.11 frames encapsulated inside 802.11 frames.

Is there a reason *not* to use ep_ allocation for structures passed to 
taps, rather than using fixed-size arrays and rotating through them?
_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@wireshark.org
http://www.wireshark.org/mailman/listinfo/wireshark-dev

Reply via email to