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

…kets Fixes #2528
video-dev · Feb 17, 2020 · 2360a6c · 2360a6c
1 parent 16769c8
commit 2360a6c
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/src/demux/tsdemuxer.js b/src/demux/tsdemuxer.js
@@ -995,15 +995,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) {
+          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++;
+          } 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 {