From c74e45e87cf60aec00a3b55a35ea3dc982b40f64 Mon Sep 17 00:00:00 2001
From: Marcus Brinkmann <m.brinkmann@semantics.de>
Date: Thu, 29 Sep 2016 14:54:25 +0200
Subject: [PATCH] Force reloading palette when using mmap in ImageFile.

---
 PIL/ImageFile.py                 |   3 +++
 Tests/images/multipage-mmap.tiff | Bin 0 -> 5760 bytes
 Tests/test_imagesequence.py      |   9 +++++++++
 3 files changed, 12 insertions(+)
 create mode 100644 Tests/images/multipage-mmap.tiff

diff --git a/PIL/ImageFile.py b/PIL/ImageFile.py
index cda64efcf..a66452478 100644
--- a/PIL/ImageFile.py
+++ b/PIL/ImageFile.py
@@ -170,6 +170,9 @@ class ImageFile(Image.Image):
                             self.map, self.size, d, e, o, a
                             )
                     readonly = 1
+                    # After trashing self.im, we might need to reload the palette data.
+                    if self.palette:
+                        self.palette.dirty = 1
                 except (AttributeError, EnvironmentError, ImportError):
                     self.map = None
 
diff --git a/Tests/images/multipage-mmap.tiff b/Tests/images/multipage-mmap.tiff
new file mode 100644
index 0000000000000000000000000000000000000000..162b0521ba0838422de4f1de884a341b93904b3b
GIT binary patch
literal 5760
zcmeI$39ye<7{~F?-{q=g%~A-JY$@7zE^V#|g^Ef=D@uz>$WpXNQ6xzVm8~pUT9g#p
zuJ$4!l@{Exv@4zW`oGPcVa7Dmm}dU(nRofkGtXJx^UQg_=Q+8#_3VP(Nfng}YPWEw
z3h5%LtO9%Fm6g_!Uv^q&1zk4nKP8)rh3ZyH>n_2ka;jKCJ7a>blGfvcu3D(goS=nk
zO0d@{9v-1YeMKdHD-t}4DkfX;F!Wu+fJ=nomsCn=Wt3IU9?GkrqDuC(m&&TBYH!t4
zSHnK`wV#^yS4(Ym)K$*`>O0Uu8ffTXhd9(>4tInj9pz}pIM#8F*GOYInrP|-CpyW=
zPSH%R=1$c@OQ&h2wKh(7hPK*i?@VVoTL&GT<6P(I<a`&nP-k6S<YJe&)MYN$RX5#T
z;YvMR<!aZsR!`Tt-VJ)WQEz?pb(4Pj8(^TD-C~fzZgrd6-QiAmx!XO480ud48D_Zq
zJz#{99yH2BMtj&8j~MGwk9ph^#u@KPPkGt|6HPKXO!_mPHPtlJ&G4L=W|?h{=e^)X
zFPZCQubAgm^S$PE3oNwA8{YJmx4k3JV()s-`#$iYk9_PCpITz6&n)w~<yKhf3#)wT
zD_>jf8{hiQ_tsczoqRv|(Rv$fw8>9?_KRQr=68SC>`#B$624y&{|t_Q^dm^tf99_L
zDE>RO<o>68%qzKLKF$r?i}|=9=<V?_FW8%v+By+27Yp87f-PAenY})e_$TpC_Q#pI
zKi;Wqj5^zvkL80k&L1QFZ}P|dP$AAAqq=eaI5fa5&L5Wt?!|mu9rX72n4e~2XyTu|
fG3I2#$0UD9&JQy5{2<vMXZHC~1capgk7nR+dtQ)H

literal 0
HcmV?d00001

diff --git a/Tests/test_imagesequence.py b/Tests/test_imagesequence.py
index 8c65e6358..0877bfb8a 100644
--- a/Tests/test_imagesequence.py
+++ b/Tests/test_imagesequence.py
@@ -63,5 +63,14 @@ class TestImageSequence(PillowTestCase):
             self.assert_image_equal(frame, firstFrame)
             break
 
+
+    def test_palette_mmap(self):
+        # Using mmap in ImageFile can require to reload the palette.
+        im = Image.open('Tests/images/multipage-mmap.tiff')
+        color1 = im.getpalette()[0:3]
+        im.seek(0)
+        color2 = im.getpalette()[0:3]
+        self.assertEqual(color1, color2)
+
 if __name__ == '__main__':
     unittest.main()