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.

Sign up for GitHub

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

Jump to bottom

Wrong allocation size #2

Closed
sincos2854 opened this issue May 2, 2023 · 3 comments
Closed

Wrong allocation size #2

sincos2854 opened this issue May 2, 2023 · 3 comments

Comments

@sincos2854
Copy link
Contributor

I don't think it will have any impact, but I am reporting it just in case.

LocalAlloc should use sizeof(BITMAPINFO), but sizeof(BITMAPINFOHEADER) is used.
BITMAPINFO is correct in Susie plug-in specification.
An access violation may occur in applications that use Susie plug-in.
However, this problem does not occur because the bit count of the output image is 32.
For all Susie plug-ins.

If you're not too busy, I think it would be nice to have this fixed.
Thank you very much.
@sincos2854
Copy link
Contributor Author

I fixed the following 5 repositories.
Thank you for accepting my pull requests.

ifpng
ifjpeg-turbo
ifwebp
ifavif
ifjxl

Could you please build and release them?

@uyjulian
Copy link
Owner

uyjulian commented Jul 8, 2023

New versions have been released for all of those.

@sincos2854
Copy link
Contributor Author

Thank you very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@uyjulian @sincos2854