Merge pull request #63 from jaywilliams/patch-1

Fix Division By Zero Warning for Focal Length

Author: Miljar

lib/PHPExif/Mapper/Native.php

@@ -165,7 +165,12 @@ public function mapRawData(array $data)
                     break;
                 case self::FOCALLENGTH:
                     $parts = explode('/', $value);
-                    $value = (int) reset($parts) / (int) end($parts);
+                    // Avoid division by zero if focal length is invalid
+                    if (end($parts) == '0') {
+                        $value = 0;
+                    } else {
+                        $value = (int) reset($parts) / (int) end($parts);
+                    }
                     break;
                 case self::XRESOLUTION:
                 case self::YRESOLUTION:

tests/PHPExif/Mapper/NativeMapperTest.php

@@ -138,6 +138,21 @@ public function testMapRawDataCorrectlyFormatsFocalLength()
         $this->assertEquals(6, reset($mapped));
     }
 
+    /**
+     * @group mapper
+     * @covers \PHPExif\Mapper\Native::mapRawData
+     */
+    public function testMapRawDataCorrectlyFormatsFocalLengthDivisionByZero()
+    {
+        $rawData = array(
+            \PHPExif\Mapper\Native::FOCALLENGTH => '1/0',
+        );
+
+        $mapped = $this->mapper->mapRawData($rawData);
+
+        $this->assertEquals(0, reset($mapped));
+    }
+
     /**
      * @group mapper
      * @covers \PHPExif\Mapper\Native::mapRawData