Bug 1396794 - Removed aPacket and aLength change draft
authorBharat Raghunathan <bharatraghunthan9767@gmail.com>
Sat, 16 Sep 2017 07:18:56 +0530
changeset 665473 debd9ed16e9a83c7713a643bde7bf41266153ac5
parent 618027 2fba314d7de77ad8ab693a2ea0112c0cda5dd564
child 731793 b230bf89e306280aebf9b5d2cbb33f4b2de33262
push id80073
push userbmo:bharatraghunthan9767@gmail.com
push dateFri, 15 Sep 2017 14:01:35 +0000
bugs1396794
milestone56.0a1
Bug 1396794 - Removed aPacket and aLength change MozReview-Commit-ID: JG9vo3dbMB
dom/media/flac/FlacFrameParser.cpp
--- a/dom/media/flac/FlacFrameParser.cpp
+++ b/dom/media/flac/FlacFrameParser.cpp
@@ -71,18 +71,16 @@ FlacFrameParser::DecodeHeaderBlock(const
 
   mPacketCount++;
 
   if (aPacket[0] == 'f') {
     if (mPacketCount != 1 || memcmp(br.Read(4), "fLaC", 4) ||
         br.Remaining() != FLAC_STREAMINFO_SIZE + 4) {
       return false;
     }
-    aPacket += 4;
-    aLength -= 4;
   }
   uint8_t blockHeader = br.ReadU8();
   // blockType is a misnomer as it could indicate here either a packet type
   // should it points to the start of a Flac in Ogg metadata, or an actual
   // block type as per the flac specification.
   uint32_t blockType = blockHeader & 0x7f;
   bool lastBlock = blockHeader & 0x80;