| Index: third_party/libwebp/dec/buffer.c
|
| diff --git a/third_party/libwebp/dec/buffer.c b/third_party/libwebp/dec/buffer.c
|
| index 1e852efe7470ea34f1918b2828093d3340918b71..42feac74c43e08b1fae732374ad4796fc34a44d9 100644
|
| --- a/third_party/libwebp/dec/buffer.c
|
| +++ b/third_party/libwebp/dec/buffer.c
|
| @@ -42,29 +42,34 @@ static VP8StatusCode CheckDecBuffer(const WebPDecBuffer* const buffer) {
|
| ok = 0;
|
| } else if (!WebPIsRGBMode(mode)) { // YUV checks
|
| const WebPYUVABuffer* const buf = &buffer->u.YUVA;
|
| - const uint64_t y_size = (uint64_t)buf->y_stride * height;
|
| - const uint64_t u_size = (uint64_t)buf->u_stride * ((height + 1) / 2);
|
| - const uint64_t v_size = (uint64_t)buf->v_stride * ((height + 1) / 2);
|
| - const uint64_t a_size = (uint64_t)buf->a_stride * height;
|
| + const int y_stride = abs(buf->y_stride);
|
| + const int u_stride = abs(buf->u_stride);
|
| + const int v_stride = abs(buf->v_stride);
|
| + const int a_stride = abs(buf->a_stride);
|
| + const uint64_t y_size = (uint64_t)y_stride * height;
|
| + const uint64_t u_size = (uint64_t)u_stride * ((height + 1) / 2);
|
| + const uint64_t v_size = (uint64_t)v_stride * ((height + 1) / 2);
|
| + const uint64_t a_size = (uint64_t)a_stride * height;
|
| ok &= (y_size <= buf->y_size);
|
| ok &= (u_size <= buf->u_size);
|
| ok &= (v_size <= buf->v_size);
|
| - ok &= (buf->y_stride >= width);
|
| - ok &= (buf->u_stride >= (width + 1) / 2);
|
| - ok &= (buf->v_stride >= (width + 1) / 2);
|
| + ok &= (y_stride >= width);
|
| + ok &= (u_stride >= (width + 1) / 2);
|
| + ok &= (v_stride >= (width + 1) / 2);
|
| ok &= (buf->y != NULL);
|
| ok &= (buf->u != NULL);
|
| ok &= (buf->v != NULL);
|
| if (mode == MODE_YUVA) {
|
| - ok &= (buf->a_stride >= width);
|
| + ok &= (a_stride >= width);
|
| ok &= (a_size <= buf->a_size);
|
| ok &= (buf->a != NULL);
|
| }
|
| } else { // RGB checks
|
| const WebPRGBABuffer* const buf = &buffer->u.RGBA;
|
| - const uint64_t size = (uint64_t)buf->stride * height;
|
| + const int stride = abs(buf->stride);
|
| + const uint64_t size = (uint64_t)stride * height;
|
| ok &= (size <= buf->size);
|
| - ok &= (buf->stride >= width * kModeBpp[mode]);
|
| + ok &= (stride >= width * kModeBpp[mode]);
|
| ok &= (buf->rgba != NULL);
|
| }
|
| return ok ? VP8_STATUS_OK : VP8_STATUS_INVALID_PARAM;
|
| @@ -131,9 +136,35 @@ static VP8StatusCode AllocateBuffer(WebPDecBuffer* const buffer) {
|
| return CheckDecBuffer(buffer);
|
| }
|
|
|
| +VP8StatusCode WebPFlipBuffer(WebPDecBuffer* const buffer) {
|
| + if (buffer == NULL) {
|
| + return VP8_STATUS_INVALID_PARAM;
|
| + }
|
| + if (WebPIsRGBMode(buffer->colorspace)) {
|
| + WebPRGBABuffer* const buf = &buffer->u.RGBA;
|
| + buf->rgba += (buffer->height - 1) * buf->stride;
|
| + buf->stride = -buf->stride;
|
| + } else {
|
| + WebPYUVABuffer* const buf = &buffer->u.YUVA;
|
| + const int H = buffer->height;
|
| + buf->y += (H - 1) * buf->y_stride;
|
| + buf->y_stride = -buf->y_stride;
|
| + buf->u += ((H - 1) >> 1) * buf->u_stride;
|
| + buf->u_stride = -buf->u_stride;
|
| + buf->v += ((H - 1) >> 1) * buf->v_stride;
|
| + buf->v_stride = -buf->v_stride;
|
| + if (buf->a != NULL) {
|
| + buf->a += (H - 1) * buf->a_stride;
|
| + buf->a_stride = -buf->a_stride;
|
| + }
|
| + }
|
| + return VP8_STATUS_OK;
|
| +}
|
| +
|
| VP8StatusCode WebPAllocateDecBuffer(int w, int h,
|
| const WebPDecoderOptions* const options,
|
| WebPDecBuffer* const out) {
|
| + VP8StatusCode status;
|
| if (out == NULL || w <= 0 || h <= 0) {
|
| return VP8_STATUS_INVALID_PARAM;
|
| }
|
| @@ -160,8 +191,17 @@ VP8StatusCode WebPAllocateDecBuffer(int w, int h,
|
| out->width = w;
|
| out->height = h;
|
|
|
| - // Then, allocate buffer for real
|
| - return AllocateBuffer(out);
|
| + // Then, allocate buffer for real.
|
| + status = AllocateBuffer(out);
|
| + if (status != VP8_STATUS_OK) return status;
|
| +
|
| +#if WEBP_DECODER_ABI_VERSION > 0x0203
|
| + // Use the stride trick if vertical flip is needed.
|
| + if (options != NULL && options->flip) {
|
| + status = WebPFlipBuffer(out);
|
| + }
|
| +#endif
|
| + return status;
|
| }
|
|
|
| //------------------------------------------------------------------------------
|
| @@ -178,8 +218,9 @@ int WebPInitDecBufferInternal(WebPDecBuffer* buffer, int version) {
|
|
|
| void WebPFreeDecBuffer(WebPDecBuffer* buffer) {
|
| if (buffer != NULL) {
|
| - if (!buffer->is_external_memory)
|
| - free(buffer->private_memory);
|
| + if (!buffer->is_external_memory) {
|
| + WebPSafeFree(buffer->private_memory);
|
| + }
|
| buffer->private_memory = NULL;
|
| }
|
| }
|
|
|