python-pillow/Pillow
1
1
Fork 0
mirror of https://github.com/python-pillow/Pillow.git synced 2024-09-21 11:28:58 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fixed endian handling for I;16 getextrema

Browse Source
This commit is contained in:
Andrew Murray 2020-02-29 20:29:28 +11:00
parent b73e3dddcc
commit 42a0653097
2 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Tests/test_image_getextrema.py
View File

@ -1,10 +1,8 @@
import pytest
from PIL import Image from PIL import Image
from .helper import hopper, is_big_endian, on_ci from .helper import hopper
@pytest.mark.xfail(is_big_endian() and on_ci(), reason="Fails on big-endian")
def test_extrema(): def test_extrema():
def extrema(mode): def extrema(mode):
return hopper(mode).getextrema() return hopper(mode).getextrema()
@ -20,7 +18,6 @@ def test_extrema():
assert extrema("I;16") == (1, 255) assert extrema("I;16") == (1, 255)
@pytest.mark.xfail(is_big_endian() and on_ci(), reason="Fails on big-endian")
def test_true_16(): def test_true_16():
with Image.open("Tests/images/16_bit_noise.tif") as im: with Image.open("Tests/images/16_bit_noise.tif") as im:
assert im.mode == "I;16" assert im.mode == "I;16"

14
src/libImaging/GetBBox.c
View File

@ -166,11 +166,21 @@ ImagingGetExtrema(Imaging im, void *extrema)
case IMAGING_TYPE_SPECIAL: case IMAGING_TYPE_SPECIAL:
if (strcmp(im->mode, "I;16") == 0) { if (strcmp(im->mode, "I;16") == 0) {
UINT16 v; UINT16 v;
memcpy(&v, *im->image8, sizeof(v)); UINT8* pixel = *im->image8;
#ifdef WORDS_BIGENDIAN
v = pixel[0] + (pixel[1] << 8);
#else
memcpy(&v, pixel, sizeof(v));
#endif
imin = imax = v; imin = imax = v;
for (y = 0; y < im->ysize; y++) { for (y = 0; y < im->ysize; y++) {
for (x = 0; x < im->xsize; x++) { for (x = 0; x < im->xsize; x++) {
memcpy(&v, im->image[y] + x * sizeof(v), sizeof(v)); pixel = im->image[y] + x * sizeof(v);
#ifdef WORDS_BIGENDIAN
v = pixel[0] + (pixel[1] << 8);
#else
memcpy(&v, pixel, sizeof(v));
#endif
if (imin > v) if (imin > v)
imin = v; imin = v;
else if (imax < v) else if (imax < v)