Handle rational tags with a zero denominator

python-pillow · Oct 16, 2024 · c2ada0c · c2ada0c
1 parent 11c654c
commit c2ada0c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/Tests/images/exif_gps_zero_denominator.jpg b/Tests/images/exif_gps_zero_denominator.jpg
diff --git a/Tests/test_file_jpeg.py b/Tests/test_file_jpeg.py
@@ -406,6 +406,10 @@ def test_exif_gps_typeerror(self) -> None:
             # Should not raise a TypeError
             im._getexif()
 
+    def test_exif_gps_zero_denominator(self) -> None:
+        with Image.open("Tests/images/exif_gps_zero_denominator.jpg") as im:
+            im.getexif().tobytes()
+
     def test_progressive_compat(self) -> None:
         im1 = self.roundtrip(hopper())
         assert not im1.info.get("progressive")

diff --git a/src/PIL/TiffImagePlugin.py b/src/PIL/TiffImagePlugin.py
@@ -294,7 +294,7 @@ def _accept(prefix: bytes) -> bool:
 def _limit_rational(
     val: float | Fraction | IFDRational, max_val: int
 ) -> tuple[IntegralLike, IntegralLike]:
-    inv = abs(float(val)) > 1
+    inv = abs(val) > 1
     n_d = IFDRational(1 / val if inv else val).limit_rational(max_val)
     return n_d[::-1] if inv else n_d