Image.load() does not close file pointer

[arve0]

I'm doing this:

from PIL import Image
img = Image.open(path)
fp = img.fp
img.load()
fp.closed # False

Expected file pointer to be closed.

System:

• Windows 7 64bit
• Python 3.4.1
• Pillow 2.7.0 (installed via conda)

[aclark4life]

It's not False, the attribute is missing:

>>> img = Image.open("/Users/alexclark/Downloads/bob 15.jpg")
>>> img.fp.closed
False
>>> img.load()
<PixelAccess object at 0x10664c110>
>>> img.fp.closed
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'NoneType' object has no attribute 'closed'

I agree this is wonky.

[arve0]

Yes, the file pointer is deleted from img object, but if you copy the reference, you can still see that its there and open.

[aclark4life]

Ah! OK marked as bug, thanks.

[radarhere]

I can replicate this on OS X, so it's not just Windows.

I wrote a patch so that the file pointer would close, but I then found that the seek method in GifImagePlugin uses the file pointer after calling load()

[QuLogic]

Same as #835?

[arve0]

@QuLogic Seems plausible. Here is the updated line nr:

Pillow/PIL/ImageFile.py

Line 246 in 6a85d51

self.fp = None # might be shared

[hugovk]

The code in the first comment now passes:

from PIL import Image
img = Image.open(path)
fp = img.fp
img.load()
fp.closed # was False in original report, but is now True

System:

• macOS Sierra
• Python 3.6.0
• Pillow 2.7.13

See also #835 (comment).