1749749 - packetsReceived in remote rtp stats can jump

Status: NEW

(Core :: WebRTC, defect)

Description

[Andreas Pehrson [:pehrsons]]

Nils reported seeing "packetsReceived" in "remote-inbound-rtp" jumping from 618 to 4294903779. This is likely due to the logic here where aRtcpData.report_block().extended_highest_sequence_number is higher than *base_seq but aRtcpData.report_block().extended_highest_sequence_number - aRtcpData.report_block().packets_lost is lower, so we go negative and wrap around the uint.

Comment 1

[Andreas Pehrson [:pehrsons]]

I'm not sure how this is possible. That code is from D108671 so perhaps Byron has an idea?

Comment 2

[Byron Campen [:bwc]]

Maybe the RTCP has garbage in it? We aren't sanity-checking these values in our code, and maybe libwebrtc isn't checking them either? I guess I'd need to see some output with MOZ_LOG=RtpLogger:5

Comment 3

[Andreas Pehrson [:pehrsons]]

Garbage (as in random) seems unlikely, 4294903779 is too close (63516) to the upper bound of uint32_t.

In fact if we offset this number by uint16_t's max (65535, aka -1), packetsReceived would be 2019 which seems entirely plausible. An RTP sequence number is 16 bits and is represented as uint16_t in various places. I looked around a bit for this in libwebrtc, but nothing stood out.

Nils, have you been able to reproduce this locally, or is it showing in your telemetry or something? Do you know what app was running the remote side?

Comment 4

[Release mgmt bot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1654112

Comment 5

[Byron Campen [:bwc]]

(In reply to Andreas Pehrson [:pehrsons] from comment #3)

Garbage (as in random) seems unlikely, 4294903779 is too close (63516) to the upper bound of uint32_t.

By "garbage", I meant just incorrect values. It is hard to say without any sort of logging or pcap.

Comment 6

[Nils Ohlmeier [:drno]]

I noticed this in stats recorded on a Jitsi call between two Firefox tabs on the same laptop. I can try to repro.

Comment 7

[Dianna Smith [:diannaS]]

do you happen to know the user impact on this?

Comment 8

[Andreas Pehrson [:pehrsons]]

End user impact should be low to none, but webrtc services may see weird data and so won't be able to carry out proper analysis of their network stats.

Comment 9

[Nils Ohlmeier [:drno]]

Declaring NI bankruptcy.