Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(950)

Side by Side Diff: third_party/libopenjpeg20/0023-j2k_merge_ppt_leak.patch

Issue 2353463002: Fix memory leak in opj_j2k_merge_ppt.
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | third_party/libopenjpeg20/README.pdfium » ('j') | third_party/libopenjpeg20/j2k.c » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 diff --git a/third_party/libopenjpeg20/j2k.c b/third_party/libopenjpeg20/j2k.c
2 index 6346c21..7712d38 100644
3 --- a/third_party/libopenjpeg20/j2k.c
4 +++ b/third_party/libopenjpeg20/j2k.c
5 @@ -3807,7 +3807,10 @@ static OPJ_BOOL opj_j2k_merge_ppt(opj_tcp_t *p_tcp, opj_e vent_mgr_t * p_manager)
6 /* preconditions */
7 assert(p_tcp != 00);
8 assert(p_manager != 00);
9 - assert(p_tcp->ppt_buffer == NULL);
10 +
11 + if(p_tcp->ppt_buffer != NULL) {
12 + return OPJ_FALSE;
13 + }
14
15 if (p_tcp->ppt == 0U) {
16 return OPJ_TRUE;
OLDNEW
« no previous file with comments | « no previous file | third_party/libopenjpeg20/README.pdfium » ('j') | third_party/libopenjpeg20/j2k.c » ('J')

Powered by Google App Engine
This is Rietveld 408576698