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

9000x9000 PNG gets corrupted #168

Closed
joshua-stone opened this issue Feb 17, 2019 · 2 comments
Closed

9000x9000 PNG gets corrupted #168

joshua-stone opened this issue Feb 17, 2019 · 2 comments
Labels
I-High Issues that are breaking core functionality for a small number of users T-Bug Some piece of the software is not working as intended

Comments

@joshua-stone
Copy link

I tried compressing a mosaic image of the moon found on Reddit, and this is what I got:

$ time oxipng -Z -o6 -i 0 Copy\ of\ updated\ mosaic.png --out moon.png
Processing: Copy of updated mosaic.png
    9000x9000 pixels, PNG format
    3x16 bits/pixel, RGB
    IDAT size = 304863576 bytes
    File size = 304863698 bytes
Trying: 6 combinations
Found better combination:
    zc = 0  zs = 0  f = 4        290407972 bytes
    IDAT size = 290407972 bytes (14455604 bytes decrease)
    file size = 290408094 bytes (14455604 bytes = 4.74% decrease)
The resulting image is corrupted and will not be outputted.
This is a bug! Please report it at https://github.com/shssoichiro/oxipng/issues
The resulting image is corrupted

real	20m33.361s
user	112m21.149s
sys	0m19.084s

Here's the latest direct download link to the PNG that's still up:

http://test.nyy.ca/mosaic.zip

@shssoichiro
Copy link
Owner

Thanks for reporting this! I'll get this fixed ASAP.

@shssoichiro shssoichiro added T-Bug Some piece of the software is not working as intended I-High Issues that are breaking core functionality for a small number of users labels Feb 18, 2019
@shssoichiro
Copy link
Owner

Looks like this and #167 were fixed upstream in image-rs/image#860. oxipng will temporarily use the fixed, unpublished version of the image crate to fix this issue.

shssoichiro added a commit that referenced this issue Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-High Issues that are breaking core functionality for a small number of users T-Bug Some piece of the software is not working as intended
Projects
None yet
Development

No branches or pull requests

2 participants