Fixing issues when inconsistent size information on ftyp boxes caused…

… MP4 parser to throw errors instead of rejecting the files

lib/parsers/mp4_parser.rb

@@ -67,7 +67,7 @@ def call(io)
   private
 
   def file_format(box_tree)
-    major_brand = box_tree.find { |box| box.type == 'ftyp' }&.fields[:major_brand]
+    major_brand = box_tree.find { |box| box.type == 'ftyp' }&.fields&.[](:major_brand)
     BRAND_FORMATS[major_brand.downcase] if major_brand
   end
 

spec/parsers/mp4_parser_spec.rb

@@ -31,6 +31,17 @@
       end
     end
 
+    context "when the 'ftyp' box can't be parsed" do
+      let(:result) do
+        path = fixtures_dir + '/MP4/invalid/inconsistent_ftyp_box.mp4'
+        subject.call(File.open(path, 'rb'))
+      end
+
+      it 'should be nil' do
+        expect(result).to be_nil
+      end
+    end
+
     Dir.glob(fixtures_dir + '/MP4/valid/video/*.*').sort.each do |path|
       context "for #{path}" do
         let(:result) { subject.call(File.open(path, 'rb')) }