From 91825ae7f53d2aea12d5b7c11138d4c1a39b0654 Mon Sep 17 00:00:00 2001 From: trav90 Date: Fri, 18 May 2018 10:02:46 -0500 Subject: [MSE] Add a missing comma in validBoxes --- dom/media/mediasource/ContainerParser.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'dom/media/mediasource/ContainerParser.cpp') diff --git a/dom/media/mediasource/ContainerParser.cpp b/dom/media/mediasource/ContainerParser.cpp index 5267e9315..594d8fc26 100644 --- a/dom/media/mediasource/ContainerParser.cpp +++ b/dom/media/mediasource/ContainerParser.cpp @@ -390,7 +390,7 @@ private: "ftyp", "moov", // init segment "pdin", "free", "sidx", // optional prior moov box "styp", "moof", "mdat", // media segment - "mfra", "skip", "meta", "meco", "ssix", "prft" // others. + "mfra", "skip", "meta", "meco", "ssix", "prft", // others. "pssh", // optional with encrypted EME, though ignored. "emsg", // ISO23009-1:2014 Section 5.10.3.3 "bloc", "uuid" // boxes accepted by chrome. -- cgit v1.2.3 From e2464b1391b986a19e8560108070c16a08ed62a8 Mon Sep 17 00:00:00 2001 From: trav90 Date: Fri, 25 May 2018 08:28:17 -0500 Subject: [MSE] Correct ADTS frame size calculation This should be bitwise OR rather than logical OR, which just returns 1. --- dom/media/mediasource/ContainerParser.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'dom/media/mediasource/ContainerParser.cpp') diff --git a/dom/media/mediasource/ContainerParser.cpp b/dom/media/mediasource/ContainerParser.cpp index 594d8fc26..4ae37d7e9 100644 --- a/dom/media/mediasource/ContainerParser.cpp +++ b/dom/media/mediasource/ContainerParser.cpp @@ -588,8 +588,8 @@ public: return false; } size_t header_length = have_crc ? 9 : 7; - size_t data_length = (((*aData)[3] & 0x03) << 11) || - (((*aData)[4] & 0xff) << 3) || + size_t data_length = (((*aData)[3] & 0x03) << 11) | + (((*aData)[4] & 0xff) << 3) | (((*aData)[5] & 0xe0) >> 5); uint8_t frames = ((*aData)[6] & 0x03) + 1; MOZ_ASSERT(frames > 0); -- cgit v1.2.3