Fix infinite loop on a corrupted video file

When the `moov` atom has the declared size more than is able to read,
the `parse` method ends up with the infinite while loop. This
modification fixes such an issue.

LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2023 Gennady Kozlenko
+Copyright (c) 2024 Gennady Kozlenko
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

lib/mp4/parser-impl.js

@@ -66,6 +66,8 @@ class ParserImpl {
                     this.moovAtom = new AtomMoov();
                     this.moovAtom.parse(buffer);
                     break;
+                } else {
+                    throw new Error('Unable to read MOOV atom');
                 }
             } else {
                 pos += atomSize;

test/mp4-parser.js

@@ -142,6 +142,18 @@ describe('MP4Parser', function () {
                 return expect(() => MP4Parser.parse(this.file)).to.throw('MOOV atom not found');
             });
         });
+
+        describe('when MOOV atom is not readable', function () {
+            before(function () {
+                this.buffer = Buffer.alloc(100);
+                this.buffer.writeUInt32BE(250, 0);
+                this.buffer.write('moov', 4);
+            });
+
+            it('should throws an error', function () {
+                return expect(() => MP4Parser.parse(this.buffer)).to.throw('Unable to read MOOV atom');
+            });
+        });
     });
 
     describe('#check', function () {