Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test failure on hppa - ERROR: TestFileIco.test_save_to_bytes OverflowError: Python int too large to convert to C ssize_t #1258

Closed
jlec opened this issue Jun 7, 2015 · 3 comments

Comments

@jlec
Copy link

jlec commented Jun 7, 2015

See downstream report https://bugs.gentoo.org/show_bug.cgi?id=551422

TestFileIco.test_save_to_bytes ... ERROR

======================================================================
ERROR: TestFileIco.test_save_to_bytes
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/Tests/test_file_ico.py", line 27, in test_save_to_bytes
    reloaded = Image.open(output)
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/build/lib/PIL/Image.py", line 2266, in open
    im = factory(fp, filename)
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/build/lib/PIL/ImageFile.py", line 97, in __init__
    self._open()
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/build/lib/PIL/IcoImagePlugin.py", line 265, in _open
    self.load()
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/build/lib/PIL/IcoImagePlugin.py", line 268, in load
    im = self.ico.getimage(self.size)
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/build/lib/PIL/IcoImagePlugin.py", line 151, in getimage
    return self.frame(i)
  File "/var/tmp/portage/dev-python/pillow-2.8.1/work/Pillow-2.8.1-python3_3/build/lib/PIL/IcoImagePlugin.py", line 161, in frame
    self.buf.seek(header['offset'])
OverflowError: Python int too large to convert to C ssize_t
@wiredfool
Copy link
Member

Can you check with this patch: #1232 ?

@aclark4life
Copy link
Member

Please reopen if still an issue after you test with patch in #1232. Thanks

@jlec
Copy link
Author

jlec commented Jun 29, 2015

Patch works here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants