Fix issue in TS Demuxer that skipped AAC frames at the end of PES pac…

…kets Fixes #2528
video-dev · Feb 17, 2020 · be895da · be895da
1 parent 12f2e16
commit be895da
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/src/demux/tsdemuxer.js b/src/demux/tsdemuxer.js
@@ -1036,15 +1036,21 @@ class TSDemuxer {
 
     // scan for aac samples
     while (offset < len) {
-      if (ADTS.isHeader(data, offset) && (offset + 5) < len) {
-        let frame = ADTS.appendFrame(track, data, offset, pts, frameIndex);
-        if (frame) {
-          // logger.log(`${Math.round(frame.sample.pts)} : AAC`);
-          offset += frame.length;
-          stamp = frame.sample.pts;
-          frameIndex++;
+      if (ADTS.isHeader(data, offset)) {
+        if ((offset + 5) < len) {
+          const frame = ADTS.appendFrame(track, data, offset, pts, frameIndex);
+          if (frame) {
+            // logger.log(`${Math.round(frame.sample.pts)} : AAC`);
+            offset += frame.length;
+            stamp = frame.sample.pts;
+            frameIndex++;
+          } else {
+            // Remaining data will be added to aacOverFlow
+            // logger.log('Unable to parse AAC frame');
+            break;
+          }
         } else {
-          // logger.log('Unable to parse AAC frame');
+          // Remaining data will be added to aacOverFlow
           break;
         }
       } else {