Changed Image size property to be read-only by default

Tests/test_image.py

@@ -56,9 +56,8 @@ def test_width_height(self):
         self.assertEqual(im.width, 1)
         self.assertEqual(im.height, 2)
 
-        im.size = (3, 4)
-        self.assertEqual(im.width, 3)
-        self.assertEqual(im.height, 4)
+        with self.assertRaises(AttributeError) as e:
+            im.size = (3, 4)
 
     def test_invalid_image(self):
         if py3:

Tests/test_imagefile.py

@@ -154,7 +154,7 @@ class MockImageFile(ImageFile.ImageFile):
     def _open(self):
         self.rawmode = 'RGBA'
         self.mode = 'RGBA'
-        self.size = (200, 200)
+        self._size = (200, 200)
         self.tile = [("MOCK", (xoff, yoff, xoff+xsize, yoff+ysize), 32, None)]
 
 

src/PIL/BlpImagePlugin.py

@@ -270,7 +270,7 @@ def _read_blp_header(self):
         self._blp_alpha_encoding, = struct.unpack("<b", self.fp.read(1))
         self._blp_mips, = struct.unpack("<b", self.fp.read(1))
 
-        self.size = struct.unpack("<II", self.fp.read(8))
+        self._size = struct.unpack("<II", self.fp.read(8))
 
         if self.magic == b"BLP1":
             # Only present for BLP1

src/PIL/BmpImagePlugin.py

@@ -143,7 +143,7 @@ def _bitmap(self, header=0, offset=0):
                           file_info['header_size'])
         # ------------------ Special case : header is reported 40, which
         # ---------------------- is shorter than real size for bpp >= 16
-        self.size = file_info['width'], file_info['height']
+        self._size = file_info['width'], file_info['height']
         # -------- If color count was not found in the header, compute from bits
         file_info['colors'] = file_info['colors'] if file_info.get('colors', 0) else (1 << file_info['bits'])
         # -------------------------------- Check abnormal values for DOS attacks

src/PIL/BufrStubImagePlugin.py

@@ -47,7 +47,7 @@ def _open(self):
 
         # make something up
         self.mode = "F"
-        self.size = 1, 1
+        self._size = 1, 1
 
         loader = self._load()
         if loader:

src/PIL/CurImagePlugin.py

@@ -71,7 +71,7 @@ def _open(self):
         self._bitmap(i32(m[12:]) + offset)
 
         # patch up the bitmap height
-        self.size = self.size[0], self.size[1]//2
+        self._size = self.size[0], self.size[1]//2
         d, e, o, a = self.tile[0]
         self.tile[0] = d, (0, 0)+self.size, o, a
 

src/PIL/DdsImagePlugin.py

@@ -114,7 +114,7 @@ def _open(self):
         header = BytesIO(header_bytes)
 
         flags, height, width = struct.unpack("<3I", header.read(12))
-        self.size = (width, height)
+        self._size = (width, height)
         self.mode = "RGBA"
 
         pitch, depth, mipmaps = struct.unpack("<3I", header.read(12))

src/PIL/EpsImagePlugin.py

@@ -213,7 +213,7 @@ def _open(self):
         box = None
 
         self.mode = "RGB"
-        self.size = 1, 1  # FIXME: huh?
+        self._size = 1, 1  # FIXME: huh?
 
         #
         # Load EPS header
@@ -240,7 +240,7 @@ def _open(self):
                             # fields should be integers, but some drivers
                             # put floating point values there anyway.
                             box = [int(float(i)) for i in v.split()]
-                            self.size = box[2] - box[0], box[3] - box[1]
+                            self._size = box[2] - box[0], box[3] - box[1]
                             self.tile = [("eps", (0, 0) + self.size, offset,
                                           (length, box))]
                         except:
@@ -289,7 +289,7 @@ def _open(self):
                 except ValueError:
                     break
 
-                self.size = int(x), int(y)
+                self._size = int(x), int(y)
                 return
 
             s = fp.readline().strip('\r\n')
@@ -327,7 +327,7 @@ def load(self, scale=1):
             return
         self.im = Ghostscript(self.tile, self.size, self.fp, scale)
         self.mode = self.im.mode
-        self.size = self.im.size
+        self._size = self.im.size
         self.tile = []
 
     def load_seek(self, *args, **kwargs):

src/PIL/FitsStubImagePlugin.py

@@ -50,7 +50,7 @@ def _open(self):
 
         # make something up
         self.mode = "F"
-        self.size = 1, 1
+        self._size = 1, 1
 
         loader = self._load()
         if loader:

src/PIL/FliImagePlugin.py

@@ -54,7 +54,7 @@ def _open(self):
 
         # image characteristics
         self.mode = "P"
-        self.size = i16(s[8:10]), i16(s[10:12])
+        self._size = i16(s[8:10]), i16(s[10:12])
 
         # animation speed
         duration = i32(s[16:20])

src/PIL/FpxImagePlugin.py

@@ -81,7 +81,7 @@ def _open_index(self, index=1):
 
         # size (highest resolution)
 
-        self.size = prop[0x1000002], prop[0x1000003]
+        self._size = prop[0x1000002], prop[0x1000003]
 
         size = max(self.size)
         i = 1

src/PIL/FtexImagePlugin.py

@@ -61,7 +61,7 @@ class FtexImageFile(ImageFile.ImageFile):
     def _open(self):
         magic = struct.unpack("<I", self.fp.read(4))
         version = struct.unpack("<i", self.fp.read(4))
-        self.size = struct.unpack("<2i", self.fp.read(8))
+        self._size = struct.unpack("<2i", self.fp.read(8))
         mipmap_count, format_count = struct.unpack("<2i", self.fp.read(8))
 
         self.mode = "RGB"

src/PIL/GbrImagePlugin.py

@@ -74,7 +74,7 @@ def _open(self):
         else:
             self.mode = 'RGBA'
 
-        self.size = width, height
+        self._size = width, height
 
         self.info["comment"] = comment
 

src/PIL/GdImageFile.py

@@ -49,7 +49,7 @@ def _open(self):
             raise SyntaxError("Not a valid GD 2.x .gd file")
 
         self.mode = "L"  # FIXME: "P"
-        self.size = i16(s[2:4]), i16(s[4:6])
+        self._size = i16(s[2:4]), i16(s[4:6])
 
         trueColor = i8(s[6])
         trueColorOffset = 2 if trueColor else 0

src/PIL/GifImagePlugin.py

@@ -65,7 +65,7 @@ def _open(self):
             raise SyntaxError("not a GIF file")
 
         self.info["version"] = s[:6]
-        self.size = i16(s[6:]), i16(s[8:])
+        self._size = i16(s[6:]), i16(s[8:])
         self.tile = []
         flags = i8(s[10])
         bits = (flags & 7) + 1

src/PIL/GribStubImagePlugin.py

@@ -48,7 +48,7 @@ def _open(self):
 
         # make something up
         self.mode = "F"
-        self.size = 1, 1
+        self._size = 1, 1
 
         loader = self._load()
         if loader:

src/PIL/Hdf5StubImagePlugin.py

@@ -47,7 +47,7 @@ def _open(self):
 
         # make something up
         self.mode = "F"
-        self.size = 1, 1
+        self._size = 1, 1
 
         loader = self._load()
         if loader:

src/PIL/IcnsImagePlugin.py

@@ -265,13 +265,33 @@ class IcnsImageFile(ImageFile.ImageFile):
     def _open(self):
         self.icns = IcnsFile(self.fp)
         self.mode = 'RGBA'
+        self.info['sizes'] = self.icns.itersizes()
         self.best_size = self.icns.bestsize()
         self.size = (self.best_size[0] * self.best_size[2],
                      self.best_size[1] * self.best_size[2])
-        self.info['sizes'] = self.icns.itersizes()
         # Just use this to see if it's loaded or not yet.
         self.tile = ('',)
 
+    @property
+    def size(self):
+        return self._size
+
+    @size.setter
+    def size(self, value):
+        infoSize = value
+        if not infoSize in self.info['sizes'] and len(infoSize) == 2:
+            infoSize = (infoSize[0], infoSize[1], 1)
+        if not infoSize in self.info['sizes'] and len(infoSize) == 3 and \
+           infoSize[2] == 1:
+            simpleSizes = [(size[0] * size[2], size[1] * size[2])
+                           for size in self.info['sizes']]
+            if value in simpleSizes:
+                infoSize = self.info['sizes'][simpleSizes.index(value)]
+        if not infoSize in self.info['sizes']:
+            raise ValueError(
+                "This is not one of the allowed sizes of this image")
+        self._size = value
+
     def load(self):
         if len(self.size) == 3:
             self.best_size = self.size

src/PIL/IcoImagePlugin.py

@@ -169,7 +169,7 @@ def frame(self, idx):
             im = BmpImagePlugin.DibImageFile(self.buf)
 
             # change tile dimension to only encompass XOR image
-            im.size = (im.size[0], int(im.size[1] / 2))
+            im._size = (im.size[0], int(im.size[1] / 2))
             d, e, o, a = im.tile[0]
             im.tile[0] = d, (0, 0) + im.size, o, a
 
@@ -263,6 +263,17 @@ def _open(self):
         self.size = self.ico.entry[0]['dim']
         self.load()
 
+    @property
+    def size(self):
+        return self._size
+
+    @size.setter
+    def size(self, value):
+        if not value in self.info['sizes']:
+            raise ValueError(
+                "This is not one of the allowed sizes of this image")
+        self._size = value
+
     def load(self):
         im = self.ico.getimage(self.size)
         # if tile is PNG, it won't really be loaded yet

src/PIL/ImImagePlugin.py

@@ -196,7 +196,7 @@ def _open(self):
             raise SyntaxError("Not an IM file")
 
         # Basic attributes
-        self.size = self.info[SIZE]
+        self._size = self.info[SIZE]
         self.mode = self.info[MODE]
 
         # Skip forward to start of image data

src/PIL/Image.py

@@ -533,7 +533,7 @@ def __init__(self):
         # FIXME: turn mode and size into delegating properties?
         self.im = None
         self.mode = ""
-        self.size = (0, 0)
+        self._size = (0, 0)
         self.palette = None
         self.info = {}
         self.category = NORMAL
@@ -548,11 +548,15 @@ def width(self):
     def height(self):
         return self.size[1]
 
+    @property
+    def size(self):
+        return self._size
+
     def _new(self, im):
         new = Image()
         new.im = im
         new.mode = im.mode
-        new.size = im.size
+        new._size = im.size
         if im.mode in ('P', 'PA'):
             if self.palette:
                 new.palette = self.palette.copy()
@@ -700,7 +704,7 @@ def __setstate__(self, state):
         info, mode, size, palette, data = state
         self.info = info
         self.mode = mode
-        self.size = size
+        self._size = size
         self.im = core.new(mode, size)
         if mode in ("L", "P") and palette:
             self.putpalette(palette)
@@ -2089,7 +2093,7 @@ def thumbnail(self, size, resample=BICUBIC):
 
         self.im = im.im
         self.mode = im.mode
-        self.size = size
+        self._size = size
 
         self.readonly = 0
         self.pyaccess = None

src/PIL/ImtImagePlugin.py

@@ -78,10 +78,10 @@ def _open(self):
                 k, v = m.group(1, 2)
                 if k == "width":
                     xsize = int(v)
-                    self.size = xsize, ysize
+                    self._size = xsize, ysize
                 elif k == "height":
                     ysize = int(v)
-                    self.size = xsize, ysize
+                    self._size = xsize, ysize
                 elif k == "pixel" and v == "n8":
                     self.mode = "L"
 

src/PIL/IptcImagePlugin.py

@@ -120,7 +120,7 @@ def _open(self):
             self.mode = "CMYK"[id]
 
         # size
-        self.size = self.getint((3, 20)), self.getint((3, 30))
+        self._size = self.getint((3, 20)), self.getint((3, 30))
 
         # compression
         try:

src/PIL/Jpeg2KImagePlugin.py

@@ -159,13 +159,13 @@ def _open(self):
         sig = self.fp.read(4)
         if sig == b'\xff\x4f\xff\x51':
             self.codec = "j2k"
-            self.size, self.mode = _parse_codestream(self.fp)
+            self._size, self.mode = _parse_codestream(self.fp)
         else:
             sig = sig + self.fp.read(8)
 
             if sig == b'\x00\x00\x00\x0cjP  \x0d\x0a\x87\x0a':
                 self.codec = "jp2"
-                self.size, self.mode = _parse_jp2_header(self.fp)
+                self._size, self.mode = _parse_jp2_header(self.fp)
             else:
                 raise SyntaxError('not a JPEG 2000 file')
 
@@ -198,8 +198,8 @@ def load(self):
         if self.reduce:
             power = 1 << self.reduce
             adjust = power >> 1
-            self.size = (int((self.size[0] + adjust) / power),
-                         int((self.size[1] + adjust) / power))
+            self._size = (int((self.size[0] + adjust) / power),
+                          int((self.size[1] + adjust) / power))
 
         if self.tile:
             # Update the reduce and layers settings

src/PIL/JpegImagePlugin.py

@@ -159,7 +159,7 @@ def SOF(self, marker):
 
     n = i16(self.fp.read(2))-2
     s = ImageFile._safe_read(self.fp, n)
-    self.size = i16(s[3:]), i16(s[1:])
+    self._size = i16(s[3:]), i16(s[1:])
 
     self.bits = i8(s[0])
     if self.bits != 8:
@@ -391,7 +391,7 @@ def draft(self, mode, size):
                 if scale >= s:
                     break
             e = e[0], e[1], (e[2]-e[0]+s-1)//s+e[0], (e[3]-e[1]+s-1)//s+e[1]
-            self.size = ((self.size[0]+s-1)//s, (self.size[1]+s-1)//s)
+            self._size = ((self.size[0]+s-1)//s, (self.size[1]+s-1)//s)
             scale = s
 
         self.tile = [(d, e, o, a)]
@@ -424,7 +424,7 @@ def load_djpeg(self):
                 pass
 
         self.mode = self.im.mode
-        self.size = self.im.size
+        self._size = self.im.size
 
         self.tile = []
 

src/PIL/McIdasImagePlugin.py

@@ -59,7 +59,7 @@ def _open(self):
             raise SyntaxError("unsupported McIdas format")
 
         self.mode = mode
-        self.size = w[10], w[9]
+        self._size = w[10], w[9]
 
         offset = w[34] + w[15]
         stride = w[15] + w[10]*w[11]*w[14]

src/PIL/MpegImagePlugin.py

@@ -72,7 +72,7 @@ def _open(self):
             raise SyntaxError("not an MPEG file")
 
         self.mode = "RGB"
-        self.size = s.read(12), s.read(12)
+        self._size = s.read(12), s.read(12)
 
 
 # --------------------------------------------------------------------