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

Memory crash in free_image_data #16

Closed
zakyg opened this issue Feb 24, 2015 · 2 comments
Closed

Memory crash in free_image_data #16

zakyg opened this issue Feb 24, 2015 · 2 comments

Comments

@zakyg
Copy link

zakyg commented Feb 24, 2015

the CGDataProviderCreateWithData call uses a pointer to a WebPDecoderConfig struct on static memory for the info (first) parameter. The free_image_data callback is called after imageWithWebPData ends and the WebPDecoderConfig goes out of scope and becomes invalid. The callback attempts to release the invalid struct and crashes the application. This issue is addressed in https://github.com/imojiengineering/iOS-WebP/commit/aa4cffa97e586e6c28d9acfb6b13e64a621dc719.
@wangkun611
Copy link

please create pull request

@brandons brandons mentioned this issue May 20, 2015
Merged
@seanooi seanooi closed this as completed Jun 9, 2015
cosmin-gordea added a commit to luberda-molinet/WebP.Touch that referenced this issue Sep 23, 2015
@cosmin-gordea
included fix for seanooi/iOS-WebP#16 (Memory crash in free_image_data…
057ab88 
…), as can be found in seanooi/iOS-WebP@c8c7fb2
@lvyile
Copy link

lvyile commented Oct 29, 2015

thx..... I used Release 0.4,
I debug for this issue hours....
somehow, I found this fix...
maybe, we can release 0.5 ...

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
4 participants
@lvyile @zakyg @seanooi @wangkun611