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

Repair of CVE-2019-6129 #269 #293

Closed
wants to merge 1 commit into from

Conversation

tangyaofang
Copy link
Contributor

Repair of CVE-2019-6129 #269

halstead pushed a commit to openembedded/openembedded-core that referenced this pull request Mar 25, 2020
Fix memory leak in png_create_info_struct.

Upstream-Status: Submitted [pnggroup/libpng#293]
CVE: CVE-2019-6129

Signed-off-by: Sakib Sajal <sakib.sajal@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
@heaven-hq
Copy link

@ctruta @ctruta I have also verified this PR. When an abnormal situation is entered, it will enter
static void display_clean_read(struct display *dp)

if (dp->read_pp != NULL)
      png_destroy_read_struct(&dp->read_pp, NULL, NULL);

branch and cause memory leak.This patch could solve this problem very well. Do you plan to merge this PR?

@ctruta
Copy link
Member

ctruta commented Nov 15, 2022

@jbowler could you please review this fix as well? The commit does seem to solve the problem, but I am not sure it's the cleanest solution, unless you confirm that it is.

@jbowler
Copy link
Contributor

jbowler commented Nov 16, 2022

NOTE: this is a duplicate of #269, I'll put comments in the original PR; this should be closed.

@ctruta
Copy link
Member

ctruta commented Nov 28, 2022

A complete fix was provided by the original author, and integrated into the master branch.

@ctruta ctruta closed this Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants