tcpdump mailing list archives

Re: [tcpdump] After setjmp/longjmp update


From: Francois-Xavier Le Bail via tcpdump-workers <tcpdump-workers () lists tcpdump org>
Date: Wed, 23 Sep 2020 13:25:11 +0200

--- Begin Message --- From: Francois-Xavier Le Bail <devel.fx.lebail () orange fr>
Date: Wed, 23 Sep 2020 13:25:11 +0200
On 20/09/2020 18:25, Michael Richardson wrote:

Also, please confirm for me that these lines like this are redundant:

      ND_TCHECK_LEN(p, IEEE802_11_REASON_LEN);
      if (length < IEEE802_11_REASON_LEN)
              goto trunc;

In fact, we don't need/want either of them anymore, because the GET_* will
handle the range check.

Answer here:
https://github.com/the-tcpdump-group/tcpdump/pull/875#pullrequestreview-493054552
(point 4)

--- End Message ---
_______________________________________________
tcpdump-workers mailing list
tcpdump-workers () lists tcpdump org
https://lists.sandelman.ca/mailman/listinfo/tcpdump-workers

Current thread: