Merge pull request #4823 from radarhere/close

Do not try to close file pointer if file pointer is empty

Tests/test_image_load.py

@@ -1,3 +1,4 @@
+import logging
 import os
 
 import pytest
@@ -23,6 +24,14 @@ def test_close():
         im.getpixel((0, 0))
 
 
+def test_close_after_load(caplog):
+    im = Image.open("Tests/images/hopper.gif")
+    im.load()
+    with caplog.at_level(logging.DEBUG):
+        im.close()
+    assert len(caplog.records) == 0
+
+
 def test_contextmanager():
     fn = None
     with Image.open("Tests/images/hopper.gif") as im:

src/PIL/Image.py

@@ -594,7 +594,8 @@ class Image:
         try:
             if hasattr(self, "_close__fp"):
                 self._close__fp()
-            self.fp.close()
+            if self.fp:
+                self.fp.close()
             self.fp = None
         except Exception as msg:
             logger.debug("Error closing: %s", msg)