Wireshark-dev: Re: [Wireshark-dev] Do we see false positives on the prechecks in merge-request

From: Pascal Quantin <pascal@xxxxxxxxxxxxx>
Date: Mon, 1 Feb 2021 16:22:04 +0100
Hi Richard,

Le lun. 1 févr. 2021 à 16:09, Richard Sharpe <realrichardsharpe@xxxxxxxxx> a écrit :
Hi folks,

In one of the builds for my merge request around Robust AV Streaming,
I got this:

https://gitlab.com/wireshark/wireshark/-/jobs/998246552

It is complaining that an ENC_BIG_ENDIAN should be ENC_NA, but in
looking at the code, it already is ENC_NA and I have not changed that
for a while (but perhaps the original code was wrong.)

This is not a false positive: your MR https://gitlab.com/wireshark/wireshark/-/merge_requests/1885 is adding those two new fields hf_ieee80211_tclas4_ipv6_src and hf_ieee80211_tclas4_ipv6_dst and they use ENC_BIG_ENDIAN instead of ENC_NA.

Best regards,
Pascal.