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

Rotate() results in panic on some images #60

Closed
cheston97 opened this issue Aug 14, 2018 · 4 comments
Closed

Rotate() results in panic on some images #60

cheston97 opened this issue Aug 14, 2018 · 4 comments

Comments

@cheston97
Copy link

I've encountered some images for which the Rotate() function results in:
runtime error: slice bounds out of range

The error seems to be happening on rotate.go:111:
copy(dst.Pix[dstPos:dstPos+4], src.Pix[srcPos:srcPos+4])

An image that can reproduce the error: https://static.thumbtackstatic.com/pictures/61/41zcu25h98b0rquu.jpg

FWIW, opening that image in an editor (e.g. Preview on mac) and simply re-saving it seems to resolve the issue. Unsure what to make of that.

Please let me know if I can provide any more info!

@ghost
Copy link

ghost commented Apr 27, 2019

I would like to look into this, if its okay ?

@ghost
Copy link

ghost commented Apr 27, 2019

Short follow up, are there any other images that seem not to work ? Because given image: https://static.thumbtackstatic.com/pictures/61/41zcu25h98b0rquu.jpg works just fine on my mac.

@cheston97
Copy link
Author

Sorry, I don't have any others available, and we migrated our image manipulation to another library, so I don't have an easy way to try to repro. Sorry!

@anthonynsimon
Copy link
Owner

No worries :)

I guess we can close the issue for now, at least until we can find a case that reproduces it.

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

No branches or pull requests

2 participants