Skip to content

Commit 94bf86b

Browse files
marconeandi34
authored andcommitted
resolve merge conflicts of 79cf158 to mnc-dev
AOSP-Change-Id: Ied32e83215e386c801c02991a0b2fa4baa25b643 CVE-2017-0558 (cherry picked from commit 50358a8) Change-Id: Ic2e40c7d6aec8427444a1fd145726e490e994d08
1 parent 508a00c commit 94bf86b

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

media/libstagefright/wifi-display/rtp/RTPSender.cpp

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -766,9 +766,15 @@ status_t RTPSender::parseTSFB(const uint8_t *data, size_t size) {
766766
}
767767

768768
status_t RTPSender::parseAPP(const uint8_t *data, size_t size) {
769-
if (!memcmp("late", &data[8], 4)) {
770-
int64_t avgLatencyUs = (int64_t)U64_AT(&data[12]);
771-
int64_t maxLatencyUs = (int64_t)U64_AT(&data[20]);
769+
static const size_t late_offset = 8;
770+
static const char late_string[] = "late";
771+
static const size_t avgLatencyUs_offset = late_offset + sizeof(late_string) - 1;
772+
static const size_t maxLatencyUs_offset = avgLatencyUs_offset + sizeof(int64_t);
773+
774+
if ((size >= (maxLatencyUs_offset + sizeof(int64_t)))
775+
&& !memcmp(late_string, &data[late_offset], sizeof(late_string) - 1)) {
776+
int64_t avgLatencyUs = (int64_t)U64_AT(&data[avgLatencyUs_offset]);
777+
int64_t maxLatencyUs = (int64_t)U64_AT(&data[maxLatencyUs_offset]);
772778

773779
sp<AMessage> notify = mNotify->dup();
774780
notify->setInt32("what", kWhatInformSender);

0 commit comments

Comments
 (0)