-
Notifications
You must be signed in to change notification settings - Fork 18k
image/png: invalid format: palette index out of range #4319
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
Labels
Milestone
Comments
Labels changed: added priority-soon, removed priority-triage. Owner changed to @nigeltao. Status changed to Accepted. |
then how do we draw the line between acceptable and unacceptable images? there will be tons of type of invalid images, how should we treat them? i argue that whatever we do with them, the user needs a control, and that unnecessarily complicates the API. i think if you want to write an image editor, and specifically want to deal with bad images, you definitely should apply an extra correction step to them before feeding them to the image package. that said, if we provide an API to retrieve the metadata of an image without parsing them, we can accept images with this kind of defeats. will let nigel to decide. |
This issue was closed by revision de6bf20. Status changed to Fixed. |
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
by darkgray:
Attachments:
The text was updated successfully, but these errors were encountered: