diff options
| author | Lei Zhang <thestig@chromium.org> | 2015-04-23 13:31:55 -0700 |
|---|---|---|
| committer | Lei Zhang <thestig@chromium.org> | 2015-04-23 13:31:55 -0700 |
| commit | 54605bea2fce2a85e6995eb7b844016f3779a205 (patch) | |
| tree | 79e752b7eefdbf3d4e42730b306a3cb2c0b2c1e4 /core | |
| parent | 6da351dd1890b465d660e6505d1b73ef3cc32558 (diff) | |
| download | pdfium-54605bea2fce2a85e6995eb7b844016f3779a205.tar.xz | |
Fix a crashier due to images with abnormal size
BUG=453553
R=thestig@chromium.org, tsepez@chromium.org
Review URL: https://codereview.chromium.org/1093323003
(cherry picked from commit ba920211c9569a38a4494398ac271a32098d81da)
Review URL: https://codereview.chromium.org/1105683004
Diffstat (limited to 'core')
| -rw-r--r-- | core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/j2k.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/j2k.c b/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/j2k.c index f944ad1afb..73dc5ab6fd 100644 --- a/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/j2k.c +++ b/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/j2k.c @@ -8008,14 +8008,18 @@ OPJ_BOOL opj_j2k_update_image_data (opj_tcd_t * p_tcd, OPJ_BYTE * p_data, opj_im l_img_comp_dest = p_output_image->comps; for (i=0; i<l_image_src->numcomps; i++) { - /* Allocate output component buffer if necessary */ if (!l_img_comp_dest->data) { - - l_img_comp_dest->data = (OPJ_INT32*) opj_calloc(l_img_comp_dest->w * l_img_comp_dest->h, sizeof(OPJ_INT32)); - if (! l_img_comp_dest->data) { - return OPJ_FALSE; - } + OPJ_UINT32 width = l_img_comp_dest->w; + OPJ_UINT32 height = l_img_comp_dest->h; + const OPJ_UINT32 MAX_SIZE = UINT32_MAX / sizeof(OPJ_INT32); + if (height == 0 || width > MAX_SIZE / height) { + return OPJ_FALSE; + } + l_img_comp_dest->data = (OPJ_INT32*) opj_calloc(width * height, sizeof(OPJ_INT32)); + if (!l_img_comp_dest->data) { + return OPJ_FALSE; + } } /* Copy info from decoded comp image to output image */ |