zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

PSFB without REMB magic word is not an error

Browse Source 
Several PSFB messages might be supported, distinguished using
the unique identifier. If the unique identifier is not REMB, it's
not an error, and so a warning should not be issued.

Bug: webrtc:10226
Change-Id: I5e79b473bd54cf0964f19329efb33354f63f5d5e
Reviewed-on: https://webrtc-review.googlesource.com/c/118686
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Elad Alon <eladalon@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26359}
This commit is contained in:
Elad Alon
2019-01-21 13:44:23 +01:00
committed by Commit Bot
parent 18cf2383aa
commit 3fdf90d621
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
modules/rtp_rtcp/source/rtcp_packet/remb.cc
View File

@ -57,7 +57,7 @@ bool Remb::Parse(const CommonHeader& packet) {
} }
const uint8_t* const payload = packet.payload(); const uint8_t* const payload = packet.payload();
if (kUniqueIdentifier != ByteReader<uint32_t>::ReadBigEndian(&payload[8])) { if (kUniqueIdentifier != ByteReader<uint32_t>::ReadBigEndian(&payload[8])) {
RTC_LOG(LS_WARNING) << "REMB identifier not found, not a REMB packet."; RTC_LOG(LS_INFO) << "REMB identifier not found, not a REMB packet.";
return false; return false;
} }
uint8_t number_of_ssrcs = payload[12]; uint8_t number_of_ssrcs = payload[12];