diff options
| author | Nicolas Pena <npm@chromium.org> | 2017-01-11 13:25:51 -0500 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-01-11 21:41:05 +0000 |
| commit | a63dd01d2fdda06f96e6188dbc3b415447bf2bc9 (patch) | |
| tree | 66cec303e475cf144b367e47d99d3a4a0c04a7c9 /third_party/libopenjpeg20/j2k.c | |
| parent | 8fa82794ffc2763e9fa1fc9d401c8e9a14d7c67f (diff) | |
| download | pdfium-a63dd01d2fdda06f96e6188dbc3b415447bf2bc9.tar.xz | |
Really fix m_nb_mct_records calculation in opj_j2k_read_mct
My previous attempt did not follow precisely the way m_nb_mcc_records
is increased in opj_j2k_read_mcc.
Previous: https://pdfium-review.googlesource.com/c/2165/
BUG=678461, 680102
Change-Id: I3e14c440e3a49b714f8cd82d44992fe647200336
Reviewed-on: https://pdfium-review.googlesource.com/2171
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'third_party/libopenjpeg20/j2k.c')
| -rw-r--r-- | third_party/libopenjpeg20/j2k.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/third_party/libopenjpeg20/j2k.c b/third_party/libopenjpeg20/j2k.c index d4dd65827c..45187e8e67 100644 --- a/third_party/libopenjpeg20/j2k.c +++ b/third_party/libopenjpeg20/j2k.c @@ -5129,6 +5129,7 @@ static OPJ_BOOL opj_j2k_read_mct ( opj_j2k_t *p_j2k, OPJ_UINT32 l_tmp; OPJ_UINT32 l_indix; opj_mct_data_t * l_mct_data; + OPJ_BOOL new_mct = OPJ_FALSE; /* preconditions */ assert(p_header_data != 00); @@ -5170,11 +5171,10 @@ static OPJ_BOOL opj_j2k_read_mct ( opj_j2k_t *p_j2k, ++l_mct_data; } - opj_mct_data_t *new_mct_records = NULL; - /* NOT FOUND */ if (i == l_tcp->m_nb_mct_records) { if (l_tcp->m_nb_mct_records == l_tcp->m_nb_max_mct_records) { + opj_mct_data_t *new_mct_records; l_tcp->m_nb_max_mct_records += OPJ_J2K_MCT_DEFAULT_NB_RECORDS; new_mct_records = (opj_mct_data_t *) opj_realloc(l_tcp->m_mct_records, l_tcp->m_nb_max_mct_records * sizeof(opj_mct_data_t)); @@ -5192,6 +5192,7 @@ static OPJ_BOOL opj_j2k_read_mct ( opj_j2k_t *p_j2k, } l_mct_data = l_tcp->m_mct_records + l_tcp->m_nb_mct_records; + new_mct = OPJ_TRUE; } if (l_mct_data->m_data) { @@ -5221,7 +5222,7 @@ static OPJ_BOOL opj_j2k_read_mct ( opj_j2k_t *p_j2k, l_mct_data->m_data_size = p_header_size; - if (new_mct_records) { + if (new_mct) { ++l_tcp->m_nb_mct_records; } return OPJ_TRUE; |