From patchwork Sun Aug 21 01:29:46 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37370 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp585608pzh; Sat, 20 Aug 2022 18:30:16 -0700 (PDT) X-Google-Smtp-Source: AA6agR7SevKK5sM+8zYVcR4lchkroH6qSujb0dGdnX7tQDWZdYdcfN5Jdtvghv0smMD9UC6QT+1X X-Received: by 2002:a05:6402:2791:b0:446:1ce0:f796 with SMTP id b17-20020a056402279100b004461ce0f796mr9773358ede.44.1661045416043; Sat, 20 Aug 2022 18:30:16 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id i21-20020a1709064fd500b00732f76a97aasi5751112ejw.899.2022.08.20.18.30.14; Sat, 20 Aug 2022 18:30:16 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=VQ99hsuS; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5D95A68B9B5; Sun, 21 Aug 2022 04:30:10 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR04-VI1-obe.outbound.protection.outlook.com (unknown [40.92.75.41]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id F254B68B984 for ; Sun, 21 Aug 2022 04:30:03 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=G4q/5t2grdIxvRtPJdrq73XxKvZopn0aoChHyk0DTkIrEInhkcTho9YTx/7tpg1IUvx++onH7zZauLkUVgr1/AY82M1Ikx3Dwb9cCQ70L22pdwupW26CiwEOZZoJSkNowAqcY2spI70rSLZrXgZzShS8m54ARrXgLjbIomLbdUTrUW8tMehncD4vM87WSKYkM9S1jMMNd3JG8FA40OyvQfZzvr4OOa9YK5tN+iHWQSeFDNWcrdP+1sABlvywiQ0RBFXkzuRN97E1vGi3jjK5o9jHcEAW1z910qXxtgRFjPeJmg9CzLRsL/rKsZxcEcGDK4hpDIV2WfRiIJQ7zhuj2Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=/b8n18hHC0b8c/TiYAK9y9z5dufBAJm7FDO4zUO7opE=; b=cCK/4PF1dwCib4SgdspxTkyy5DlZ0xCDjOrnQOf/ACSmncmDp8zI8cQ/sB//j/0qUgzFNOHg5H1XO4hyUtAK9evlzIu50GG2wf3zbxi4Cigg6PelXP33w1YEJXOwRAjDOdjXu+J154c0nCGzvDs5Ra7zMyHBlPcSKFbWIld+3BtpnAHnsE8oIChE2XJjkaPUi5Qymbz7GOQQAl8LGLMvLD5Y+Ny1OR9cNdmLVR32BGA4bmmGqWJ2v66nR4KFOxXk+86s2hKx9pLuNesq2BYAep/6cHH0CcanJoiADi1cQ2mEqoP1vMxAzpkrmK6QAOO7/Ur7RT0fkChQmZxkmsnfWA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/b8n18hHC0b8c/TiYAK9y9z5dufBAJm7FDO4zUO7opE=; b=VQ99hsuSR0zTrwA1a57VLsP01l0ixrR8Uzy6mqsYlOF6ydiqVQjEjEsas2J6UZkNq2GoCK/Q1EdFIa89vw7kpT6AXWLYWQ3BpzTdw/Fyz8VGAQhzfLu/7WijDmifcXJ0GbfSemiyFPH82Ov7OA+HRMTR/vM6eVImKNBLn9KQQzmLM+Jc0SM+eX4lylg4EFROjfEDcgeEWPKZhCLv1rr7ge6kPFGsUVx905pTeZmSsJyrh6k6ZVbTgzbk3d5/ENA7dcT/e/ehlio/b/F4DQGEWdFwxsDZQx6te2tPuO+hmbLSHbT8QP07AhYDmcbz+OsZpY1DTCU+YMEOo7t/LiX3/g== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by DB3PR0102MB3642.eurprd01.prod.exchangelabs.com (2603:10a6:8:11::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5546.16; Sun, 21 Aug 2022 01:30:01 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46%11]) with mapi id 15.20.5546.021; Sun, 21 Aug 2022 01:30:01 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sun, 21 Aug 2022 03:29:46 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 X-TMN: [Q+xhTpwVnta/LANchWUFNXz85gkrMwtk] X-ClientProxiedBy: ZR0P278CA0174.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:45::20) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <20220821012949.1345479-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 122e9f0a-223d-41b8-7907-08da8314aa62 X-MS-Exchange-SLBlob-MailProps: ZTmCor6bjd/ba3iRGhhtxD+cx0htDG6A5bgC7ESQbi70tsqqIDz2mcyVqVevAMlabjg0Rl5FnaDgiROlai3/aFbOybjrvI9GztlUBc0W6hCdXFbzkGfIblBFAqeh6SxNEN0n0T6ceq4Jo9ZDN+FpIe/snZ1hczd4bTXb4ubHJSD45F+F7oNkriHjZrjuHPvyqUE5TXY7ofcG8Ng7MUZfR2wD0rfrpiSIkrNQLPRJooajnpwNsfmhuN1SwEa/GSjqUIIPKCfPNPVMrTSFFe8VzZ7kdDiKlwazN3nnxJyOQTvbc3dzuzN/J1+ooi30jXD4/x7yQV8F/hgemb2UAh8dUZnzl9Ohye5WxaAjMxDuSOmbFg++mz0EKbgU9DZjwlIvTQmu3JOJXe3OeEg+RxXcqd3gOXIMs1BWaffIx9wCFCTjtzwBIraKKnDDDVO1aOa98TQMyblOnCVMgQTmQihkRXuJTjXbDLa62f6cEuI5nW7OXxNDc0nWLXEZfeuWAGEBGei1qKFt2FyLewz0gs0/FWSGcO92ZnS4pdaFYAmDrr87YLQDSI8tMOqXFXlq0VbtTMU7qsHVUtnSqV/lAhFd0bSGiaYjVekW+1nGwwaD5pGBruxctGmZqMlbNABeyCSG22aU2DbLW7UtIAW/OX+OaTihtdS8Mcw4kc3fdaJhya1ww/Uyl9O9RSsl917Ch6GlYvjUOaDVvYZL1XpuF+p3blHW1fAxOH6RaUS+gmWow04Gd5gjiQ3TaGJ1MZXd8reYNfiNnLWLIS6CrG6oFKv8OWmwL+JoIPjTF6S80zIud1YmD8MmJg59AkWGknLJiu5C X-MS-TrafficTypeDiagnostic: DB3PR0102MB3642:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: gidXlQkEHnV9+RPI3X67zhMq64MJnceea1p3VRf++ot5RwzeECjtmD26iXNz1n7FZwOWGCpuFvsdL64lfnfngFkGrhVFTnrdIjyJB1DlvlaOwjl3ORSiZACAjoxgn67lhrmHSBWrpiPpJKJ1BjyUdP2K0yHoDWFRCOwpH9YmZqw0Ut7Z0BRVjHqWZkpUN6miovbdtdpOET0kOCB3I21KXchWmWqEwz6uNq03gqikmOvOkJcjrpfqV9bAdYGTmBaBPTmbcMjnAPPPvHI4AS7snc6E31OozRK/Bmzzvl13TH74YaxIldmgFcmKsN/nfFUfh4TpIFOmfpnbP32pXhIc3zyB19cl1AbtW9ufWpRM2oL65Yecad0oivgHfhpttHrQtjQUrYaX5yDB8h8DC20i8xd9BMriSSRu6E8gi6X548bo26ATEUcGa0QrJ5ZBebCzEAkjgQ7cicfvDgUcBvriO50M8Zo/3+nZT/UJDcEqaR5A5XlxYfllQHy2wCZK1njSo8P34Bob0eVl4RWg92n/W92hfua+zTivHoeeBrnUUj8mC5YewOXDH9RG8YojUGhWNu+/tptXCljQlqEQPLZATEZBYRyU8GwC+j/Y2fXK6mEIX7i3nzjPngxgYtSjScRVRuRVRDvb9vRdwOdd0BgJjw== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: GQXe7nU05a7j9OpC3Y45jEVoPpkXmeBpHPqrkpivxIs+Fkw1nVZcXpuFxLx4YUPcUXVb9K/k8tK1Ru1d6IYArvnCpAMJ6baLA65CoZuQctvEdy0wglETbVWdaV96GgzKXqmX4C2bAWnOnYfcdlG3TumasxvEpxGctB7R3UBj898N/QddZwPYDofYOG/WidGAY8AyuVEEPoDA8uGnXmHhF0Am+SKUfWxxWqWp3eGkX++txMq7OidoDgUUQHrAiVW5k+77/3umZkjGQODYmjxVjFXKSfFkXPY8FXErg1fv6A/uqWezowkVa6d06/+dNnyJ5+q54+V6khV4NWMx+Kc4Ic5hWT/i5EsgMoXDDeEP6gTcndDPxWtGQjE9tQ3GIHNP8AEFp+4jg2q48dQOfj5UoXJebAvoomd4DmKp695Nz7kyUIprVkfn3MwJJKa6ARarAjfvqVsYu05bTkxFovcqhYpDIxFf5aBaHmJxWarDHziNG27Hb6E4VBfXErKvwo86d/J4MmbFzMJZda2Zx8cqMQ/tnvf9Bp/urP99M95Htvrfd8avGdjby9EgwfvyIivKTlCkiFnOWRxX3FB3620Lh7uF/dQKfEeX0CM3yzgePfZb0qjm636onrkw+k3q7mdt6U1JXO9+rWg/hwXBXp77H8/NOB2r8q3zLb4ZohMRg9GcA/rCgBE/t1TJvny4qizuq1ldvHqdWnUtwj+KrkqMXQ4N2O+nGvKx7NTSEiMa0IIQd5b7h51Mi0k8L7MaGH6/eVNhpJlNMjgJ5HGw5Rix0QoKtnfWy7gWSbn2y2TdrsrM0Hp/YEthbC/i/ZUAhIzVtX4ogs2+B33Vy0RPONASbmyBjrUPS03K/zb5mJ9CxAl+OhXxmm/9n/ZVpVI3/a4e9usRz78MzejmhHO+fa4mx3m0hoTb7l62oDP7cNSiN2OMI1DZpft+qP5Hv+NTznHTLYC/4B9Xkl5IFdWNzSYs0Io9A1nFH2M5M+EBJEQm3wbpDW7SLIGIZa0NP56GjSMmTgtgqdgJE9guLgp28l9xF3wkVGEsJ0qKUtfkANZUiOs5esuaEM4xPlLsfSIcgobjDve6WZnATj+b+p1N0nhz/i4sqLe1V0/sSyHd/1Fk+CQen3FlwdTItFnzvtKNG3HI+0GXlkAAvRYZ7m4t8CXU1fiv/sN8FsJOTVzLgsXBazQ0540iRyVsQ3GzKqO91iWR/0+hIkNwrVlt7/OJC7890MMchOokiB+mW2OjwG5YKySgS6/uoqDkuyQqn3pR8PJUedGnTjSiGtsvi0db6rtG1mIulgJir9JAAp/FiFFmljY= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 122e9f0a-223d-41b8-7907-08da8314aa62 X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Aug 2022 01:30:01.2979 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB3PR0102MB3642 Subject: [FFmpeg-devel] [PATCH 1/4] avcodec/pngdec: Fix APNG_DISPOSE_OP_BACKGROUND X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: FSSZUs2vRyCg APNG works with a single reference frame and an output frame. According to the spec, decoding APNG works by decoding the current IDAT/fdAT chunks (which decodes to a rectangular subregion of the whole image region), followed by either overwriting the region of the output frame with the newly decoded data or by blending the newly decoded data with the data from the reference frame onto the current subregion of the output frame. The remainder of the output frame is just copied from the reference frame. Then the reference frame might be left untouched (APNG_DISPOSE_OP_PREVIOUS), it might be replaced by the output frame (APNG_DISPOSE_OP_NONE) or the rectangular subregion corresponding to the just decoded frame has to be reset to black (APNG_DISPOSE_OP_BACKGROUND). The latter case is not handled correctly by our decoder: It only performs resetting the rectangle in the reference frame when decoding the next frame; and since commit b593abda6c642cb0c3959752dd235c2faf66837f it does not reset the reference frame permanently, but only temporarily (i.e. it only affects decoding the frame after the frame with APNG_DISPOSE_OP_BACKGROUND). This is a problem if the frame after the APNG_DISPOSE_OP_BACKGROUND frame uses APNG_DISPOSE_OP_PREVIOUS, because then the frame after the APNG_DISPOSE_OP_PREVIOUS frame has an incorrect reference frame. (If it is not followed by an APNG_DISPOSE_OP_PREVIOUS frame, the decoder only keeps a reference to the output frame, which is ok.) This commit fixes this by being much closer to the spec than the earlier code: Resetting the background is no longer postponed until the next frame; instead it is applied to the reference frame. Fixes ticket #9602. (For multithreaded decoding it was actually already broken since commit 5663301560d77486c7f7c03c1aa5f542fab23c24.) Signed-off-by: Andreas Rheinhardt --- Btw: If we had a function that only references a frame's data (and leaves the dst frame's side data completely untouched), we could apply the side data directly to the output frame, making 8d74baccff59192d395735036cd40a131a140391 unnecessary. I also wonder whether the handle_p_frame stuff should be moved out of decode_frame_common() and in the codec-specific code. libavcodec/pngdec.c | 98 ++++++++++++++++++++++----------------------- 1 file changed, 48 insertions(+), 50 deletions(-) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 3b888199d4..8a197d038d 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -78,11 +78,8 @@ typedef struct PNGDecContext { enum PNGImageState pic_state; int width, height; int cur_w, cur_h; - int last_w, last_h; int x_offset, y_offset; - int last_x_offset, last_y_offset; uint8_t dispose_op, blend_op; - uint8_t last_dispose_op; int bit_depth; int color_type; int compression_type; @@ -94,8 +91,6 @@ typedef struct PNGDecContext { int has_trns; uint8_t transparent_color_be[6]; - uint8_t *background_buf; - unsigned background_buf_allocated; uint32_t palette[256]; uint8_t *crow_buf; uint8_t *last_row; @@ -725,9 +720,30 @@ static int decode_idat_chunk(AVCodecContext *avctx, PNGDecContext *s, } ff_thread_release_ext_buffer(avctx, &s->picture); - if ((ret = ff_thread_get_ext_buffer(avctx, &s->picture, - AV_GET_BUFFER_FLAG_REF)) < 0) - return ret; + if (s->dispose_op == APNG_DISPOSE_OP_PREVIOUS) { + /* We only need a buffer for the current picture. */ + ret = ff_thread_get_buffer(avctx, p, 0); + if (ret < 0) + return ret; + } else if (s->dispose_op == APNG_DISPOSE_OP_BACKGROUND) { + /* We need a buffer for the current picture as well as + * a buffer for the reference to retain. */ + ret = ff_thread_get_ext_buffer(avctx, &s->picture, + AV_GET_BUFFER_FLAG_REF); + if (ret < 0) + return ret; + ret = ff_thread_get_buffer(avctx, p, 0); + if (ret < 0) + return ret; + } else { + /* The picture output this time and the reference to retain coincide. */ + if ((ret = ff_thread_get_ext_buffer(avctx, &s->picture, + AV_GET_BUFFER_FLAG_REF)) < 0) + return ret; + ret = av_frame_ref(p, s->picture.f); + if (ret < 0) + return ret; + } p->pict_type = AV_PICTURE_TYPE_I; p->key_frame = 1; @@ -985,12 +1001,6 @@ static int decode_fctl_chunk(AVCodecContext *avctx, PNGDecContext *s, return AVERROR_INVALIDDATA; } - s->last_w = s->cur_w; - s->last_h = s->cur_h; - s->last_x_offset = s->x_offset; - s->last_y_offset = s->y_offset; - s->last_dispose_op = s->dispose_op; - sequence_number = bytestream2_get_be32(gb); cur_w = bytestream2_get_be32(gb); cur_h = bytestream2_get_be32(gb); @@ -1086,23 +1096,6 @@ static int handle_p_frame_apng(AVCodecContext *avctx, PNGDecContext *s, ff_thread_await_progress(&s->last_picture, INT_MAX, 0); - // need to reset a rectangle to background: - if (s->last_dispose_op == APNG_DISPOSE_OP_BACKGROUND) { - av_fast_malloc(&s->background_buf, &s->background_buf_allocated, - src_stride * p->height); - if (!s->background_buf) - return AVERROR(ENOMEM); - - memcpy(s->background_buf, src, src_stride * p->height); - - for (y = s->last_y_offset; y < s->last_y_offset + s->last_h; y++) { - memset(s->background_buf + src_stride * y + - bpp * s->last_x_offset, 0, bpp * s->last_w); - } - - src = s->background_buf; - } - // copy unchanged rectangles from the last frame for (y = 0; y < s->y_offset; y++) memcpy(dst + y * dst_stride, src + y * src_stride, p->width * bpp); @@ -1171,6 +1164,22 @@ static int handle_p_frame_apng(AVCodecContext *avctx, PNGDecContext *s, return 0; } +static void apng_reset_background(PNGDecContext *s, const AVFrame *p) +{ + // need to reset a rectangle to black + av_unused int ret = av_frame_copy(s->picture.f, p); + const int bpp = s->color_type == PNG_COLOR_TYPE_PALETTE ? 4 : s->bpp; + const ptrdiff_t dst_stride = s->picture.f->linesize[0]; + uint8_t *dst = s->picture.f->data[0] + s->y_offset * dst_stride + bpp * s->x_offset; + + av_assert1(ret >= 0); + + for (size_t y = 0; y < s->cur_h; y++) { + memset(dst, 0, bpp * s->cur_w); + dst += dst_stride; + } +} + static int decode_frame_common(AVCodecContext *avctx, PNGDecContext *s, AVFrame *p, const AVPacket *avpkt) { @@ -1434,6 +1443,9 @@ exit_loop: goto fail; } } + if (CONFIG_APNG_DECODER && s->dispose_op == APNG_DISPOSE_OP_BACKGROUND) + apng_reset_background(s, p); + ff_thread_report_progress(&s->picture, INT_MAX, 0); return 0; @@ -1456,15 +1468,10 @@ static void clear_frame_metadata(PNGDecContext *s) av_dict_free(&s->frame_metadata); } -static int output_frame(PNGDecContext *s, AVFrame *f, - const AVFrame *src) +static int output_frame(PNGDecContext *s, AVFrame *f) { int ret; - ret = av_frame_ref(f, src); - if (ret < 0) - return ret; - if (s->iccp_data) { AVFrameSideData *sd = av_frame_new_side_data(f, AV_FRAME_DATA_ICC_PROFILE, s->iccp_data_len); if (!sd) { @@ -1515,13 +1522,12 @@ fail: } #if CONFIG_PNG_DECODER -static int decode_frame_png(AVCodecContext *avctx, AVFrame *dst_frame, +static int decode_frame_png(AVCodecContext *avctx, AVFrame *p, int *got_frame, AVPacket *avpkt) { PNGDecContext *const s = avctx->priv_data; const uint8_t *buf = avpkt->data; int buf_size = avpkt->size; - AVFrame *p = s->picture.f; int64_t sig; int ret; @@ -1555,7 +1561,7 @@ static int decode_frame_png(AVCodecContext *avctx, AVFrame *dst_frame, goto the_end; } - ret = output_frame(s, dst_frame, s->picture.f); + ret = output_frame(s, p); if (ret < 0) goto the_end; @@ -1574,12 +1580,11 @@ the_end: #endif #if CONFIG_APNG_DECODER -static int decode_frame_apng(AVCodecContext *avctx, AVFrame *dst_frame, +static int decode_frame_apng(AVCodecContext *avctx, AVFrame *p, int *got_frame, AVPacket *avpkt) { PNGDecContext *const s = avctx->priv_data; int ret; - AVFrame *p = s->picture.f; clear_frame_metadata(s); @@ -1608,7 +1613,7 @@ static int decode_frame_apng(AVCodecContext *avctx, AVFrame *dst_frame, if (!(s->pic_state & (PNG_ALLIMAGE|PNG_IDAT))) return AVERROR_INVALIDDATA; - ret = output_frame(s, dst_frame, s->picture.f); + ret = output_frame(s, p); if (ret < 0) return ret; @@ -1646,15 +1651,9 @@ static int update_thread_context(AVCodecContext *dst, const AVCodecContext *src) pdst->compression_type = psrc->compression_type; pdst->interlace_type = psrc->interlace_type; pdst->filter_type = psrc->filter_type; - pdst->cur_w = psrc->cur_w; - pdst->cur_h = psrc->cur_h; - pdst->x_offset = psrc->x_offset; - pdst->y_offset = psrc->y_offset; pdst->has_trns = psrc->has_trns; memcpy(pdst->transparent_color_be, psrc->transparent_color_be, sizeof(pdst->transparent_color_be)); - pdst->dispose_op = psrc->dispose_op; - memcpy(pdst->palette, psrc->palette, sizeof(pdst->palette)); pdst->hdr_state |= psrc->hdr_state; @@ -1705,7 +1704,6 @@ static av_cold int png_dec_end(AVCodecContext *avctx) s->last_row_size = 0; av_freep(&s->tmp_row); s->tmp_row_size = 0; - av_freep(&s->background_buf); av_freep(&s->iccp_data); av_dict_free(&s->frame_metadata);