From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id CBE064A6F1 for ; Thu, 4 Apr 2024 05:03:26 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 034CE68D168; Thu, 4 Apr 2024 08:03:09 +0300 (EEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2054.outbound.protection.outlook.com [40.92.90.54]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id C0DC268C068 for ; Thu, 4 Apr 2024 08:03:05 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FdrJkmoQ+wp9cAH6/kl0eaMrMJXgxaHLGa3B/sSI/CBxSz0jEJIBMw0TwM0WOS072jpXWHP41cWBkYKb0QoshjvmftIVWUMk0S9VbDZ3V3tZBOSYL8GNuNtscZlk6HoprwGQG41Dq1RYUg9vHl6J9HfW4dp5TdvKr6vFw2oIkYaYErliGiRzseCuNEoRIAT28KZyDhZ+g3LHs0IopKdWoR+MY3u8/psN7bT4CKczlrjimF9utnThNevfFqtyVW3w3rh+a0xHsUqfMz0FAGL7vZ5sVVOXHc9yfBySU523xSqEzqP9tzF3hYPlTveGwu4Sibbgny8aDEy4wD4i689rPg== 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=F5Qju5+OzPAs8MLX6PQ7j7XYj5yLQjjVL+I/Z3IW9vc=; b=LOjMTnpj95bEU09y5410sUrRx7oVL5kQ6lCyIi4XsSjUIKoDo/hRph1VCywL8s/VDecZ5Oc9s2ceWWRehGQ2ulL234i35MG4FvzH0bMzASxEIn66VQGhi8oda3RAbTw/DSeFvR0djt/Mw2nz3qHNVPSz/HE0WtelgZG3bpBjVMiVom6cCVRZGc/AVfWx1jWB6PAIVFnLXCiU9g5wOTzfadA8/cq9Uk7Gxza/x3akjdoJ3L6R+S0eifGOdVHeMqmPh9GOgZAiruGG8AfEFlw9u4OxgaNP4S3RoKMkt8O2DiKtvArasKHKFUVt+vSkSQdx+lxFjQVjIDznRiZAzIVuAw== 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=F5Qju5+OzPAs8MLX6PQ7j7XYj5yLQjjVL+I/Z3IW9vc=; b=fgOxzvCeJTHPcqqj4Tk8ksivvEhTgHqLg3qf2bdAHv8cMcS7hV8i9LeelYWT0wq1D3cxlbgZLeXjojCwFCO/KTuaOz91HORCwG9buG52kcicPjAmfSylloARko7TeEJMfs0LR43NjnzKOHk5lN5dVHtC8f8HcHhRM/9r9FJtDw2DaxAUxocybcNWrKkHtEhMSEn9jkvtIynFdnZ/ZRfxNZvlEW1lHnV2X1hI+U9jS652mvddX2ZrK/jFPDeMOB1JMOK2i7zTDHaLbOoN6cBBLpyRidNxrO6y0xcPbSwD5Tc+DYs1leWGBu04cOpZhC8eB5PsYjX0fCV8oSMXVWEzsQ== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DU0P250MB0436.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:348::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7409.46; Thu, 4 Apr 2024 05:03:01 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::4a3b:cf76:cddc:c68d]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::4a3b:cf76:cddc:c68d%4]) with mapi id 15.20.7409.042; Thu, 4 Apr 2024 05:03:01 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Thu, 4 Apr 2024 07:02:48 +0200 Message-ID: X-Mailer: git-send-email 2.40.1 In-Reply-To: References: X-TMN: [d18RfRST68lCAOrNbX1hJQFniXDE3czJ/hXYcgn+Zxk=] X-ClientProxiedBy: ZR2P278CA0028.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:46::8) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <20240404050250.3450360-3-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|DU0P250MB0436:EE_ X-MS-Office365-Filtering-Correlation-Id: 4dea3817-b3a5-414c-aaf8-08dc546480b0 X-MS-Exchange-SLBlob-MailProps: quCBMN2EvO+/Tg02NQSOAqMNnTlnxFBGqTCoZdMLb9RDuH+Q/LQ+P6Nyb7fV9EvtaTTYefqybzndBBoXQvdDArX0ZAucVXjznamqTZhScutg69Q44RWVS2g5wICLpZX32hgG85XhUiH2+UJ9mqKaOc+R2bGzd0SaTkKXQfQ1nSg/1zogLII2fim7qqTLItWKSsYvlDf5QDfqr2upW5Qh3i9uVd3/yu2MjiXOeawCqbdcYQN25c1IYKG+RA3AwXtRzHvQqni2hNTy+XdJUk82Y1zYCXLNyk8zlJ59E0go4zz+NvQ8ObCOlAsGiboIC8XiPOWIRWq5JS+ZoHxvfK+UXZHYyrhSc1YzouyDWuyxkBDjlj/LQlWXyZeORrxMudbtqKCchdy7Ot/kALt57oYRNO1Zt9ziYrUmZFX7I9b6eudDXviUA+VmYixhqrQbzVp6FrWhNkhrFMJT1auBdp3riElfTNrH1PyME5LOXpaRxriQK186QQq5ukYOIjCkjtdYQu3NqKA+D9YfEUXXF629an9GOIwrUT8iEJZ2B4vfPU1EStSUSHyvshUF6YLaB04rCzEd0yzJ7SxsToDYyGkkzFfkLBAQGipi5CHpIr9vmNY3ilzbylJZwsRgBdUDYMFYX2peD6cAFITMlvz4GBLFkOt5p13//WBDcs1LqY4GgeAGVflcCvlplNEFr5/SruMCKWYXfN52ZDh/AA5LWzZWHQ== X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: n43oek+q2isN0JoVGRRGZ6WU+ApAYroB/Ws6+UvLqwyK3xyjcP61BKp+1hiHb7UI1syflISDXpjA7jplj9qVVw3Kr+keYxtmkLmUsYEJdFS4ezZGXAtwLV8qMWaLy5Qn6GK7mDlTNaHolxTJo7x4929GX9b8QrMXaWklnyJ2CnmCsxguOKsK+P0hzoJmF6ZPFrAGue5iVF3jXElGuws5IL0e97+c5P2JEm+eG7QRdlFDRZ/1SA2OGfCLY4QfCXUIKWHq9nKuy6rJP3b1iO2S25EotHDRaIOV2/M0fMU4OkeEVIT4jiqyHsnIUNB6besv5F7Z0P/qFQ12Y0lvifeiWKsW4bwRmHuFRruFXoA1avLmUSp9fnoRwin5Y0P+fYD7uOwhVf3FTpSibAQEE+Esbwjhr6+bVc7X0JTtu9Dd2YfnfA7CVT2GYJ8ZaDitvHDWUdF578nVQnXkUGMRaJl5tX7pJ0pEHIXfBWMNAJsckd/KZptr0U3CMJoirLiH44kXp6zk8YdgHyRFzYWQ5jCZU7424pi6kuBschRxc/xJ32JFq3NnBvoVWwJxEFfr84gMCQgVD5SzCE7QbIWAgoMzbBw7oW69E1nxboiZ3ZPbcZpPuFTQ1cqk7o1Op3roCmq0 X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?jCApBq+WgUCFMIGfSIKuBjVjvPJ0OSE9tw8qt42XZjhkojbse+bZvzuWoNuv?= =?us-ascii?Q?mK9LFV836YI2LpbrGA6/BtiIVVZhYPP/fKXRJOTliOAoYF/HZy8TrqVRR2WI?= =?us-ascii?Q?y7pyfKlSHaNd+o7kIVjhU4QeQHkDq91Km6CRZ/D5eHPOyl6PZ4aseOZRkPws?= =?us-ascii?Q?IsHoy7E+hX4s6MxFtXO1LW5gNM/0JKtPgQellXSfUuDNcctTBZGp6cmapu1T?= =?us-ascii?Q?R8u0u+r9yeshG2Kbq9X337wep2y+65m4B3rpH4MHiNmd1jJRj26aUJ7obkUq?= =?us-ascii?Q?YUixJBBklmjJR/HFG7o4s81a7WqRyoMS8sQ0NLL+4fSMEplKW5iAajzAxYeV?= =?us-ascii?Q?NkuLecjxL3H3GojCw/KDbFwrUw25Sa/y4xQtEAc3A1FDmNp2SK8SAItLYpHE?= =?us-ascii?Q?7PWiFk/4NTizVhZNKjmP1k+Gt24Ir1bh5BSDOgnhoeocUOOQs2cBf+76MPxn?= =?us-ascii?Q?3kLx3fuK7l1td5JsfABaHwEhxImTDpL/C98MCZTiUphQdppIFzL4NrkQUBDW?= =?us-ascii?Q?U5jUrWU0W3voktRTft1f0WtMcjsXJeSo4K911MtAYVb1Gn2kRJBvtruJvZdy?= =?us-ascii?Q?BZqKfuKjS6+qecVuv8UQf4pVB5G5FfbpygeGN5/99yFeLHsKOcam62KQAudk?= =?us-ascii?Q?Jzg2m3oYvNL4sUbm9GMVfBmov57MG5iByhgomC6sZCpq/3ae6cgIRO5RIVfa?= =?us-ascii?Q?GyURnqTNnoiFnIKzZJzaMdT+mmTBWVmz2ms6Lyd5L/x0Fjy1i9vggZ5NTH1p?= =?us-ascii?Q?fHSDX5xFr59Jfl4YTKBA1cFJrl+d5ztH1b5xhofv8SAUOQmbx4UtrHPPeB0I?= =?us-ascii?Q?6jmtiDrLJMy7g/K8cfHDkYobU86pEqF1uR/4aSvuQLfieEQKCbuhkHlRF7F2?= =?us-ascii?Q?v6n/ThUobC7mFhaXjyV9t3GB/kmxD/iLGLVMTEm3vnOYZAfwbF00JgUd8M4f?= =?us-ascii?Q?bbqKlkJ+rIeweErU0rmi9kKlCaJag5oU6Z/k85O18o+SgKBs8eRn7PAVA9oN?= =?us-ascii?Q?6xTIwzRiup5JxmxnbJUSwj49FyHeevuNbuWUEehj47/Gvi86XzJSw0cks77C?= =?us-ascii?Q?BbjFq+0JEurEeAZRJKrdrDC3/RHoY8GSFPIvhD8UubySN7/eQKcsujJxr0Fb?= =?us-ascii?Q?Cnu+SxPOXEhPL2VA6ewQ9Av+o0bqW2+3Z76iNgAM6uwSPDtkaLFClKT+pYwx?= =?us-ascii?Q?7E43jVkJWQckP9DsAxaLMPLXwa6NssuJyuT9Y790qiYSz9ARplZHjA+3Lney?= =?us-ascii?Q?8shKYyXw37ngVFjmvw/pUjNZQ7/JeiOfaP7htqFe9Q=3D=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4dea3817-b3a5-414c-aaf8-08dc546480b0 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Apr 2024 05:03:01.4317 (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: DU0P250MB0436 Subject: [FFmpeg-devel] [PATCH 4/6] avcodec/huffyuv: Return proper error code 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 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Also forward said error code in the encoder. Signed-off-by: Andreas Rheinhardt --- libavcodec/huffyuv.c | 3 ++- libavcodec/huffyuvenc.c | 6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/huffyuv.c b/libavcodec/huffyuv.c index 723ab6b92b..f22c5ebc59 100644 --- a/libavcodec/huffyuv.c +++ b/libavcodec/huffyuv.c @@ -31,6 +31,7 @@ #include #include +#include "libavutil/error.h" #include "libavutil/log.h" #include "libavutil/macros.h" @@ -48,7 +49,7 @@ int ff_huffyuv_generate_bits_table(uint32_t *dst, const uint8_t *len_table, int for (int i = FF_ARRAY_ELEMS(lens) - 1; i > 0; i--) { if ((lens[i] + codes[i]) & 1) { av_log(NULL, AV_LOG_ERROR, "Error generating huffman table\n"); - return -1; + return AVERROR_INVALIDDATA; } codes[i - 1] = (lens[i] + codes[i]) >> 1; } diff --git a/libavcodec/huffyuvenc.c b/libavcodec/huffyuvenc.c index 4f709143a2..152f94cefb 100644 --- a/libavcodec/huffyuvenc.c +++ b/libavcodec/huffyuvenc.c @@ -232,9 +232,9 @@ static int store_huffman_tables(HYuvEncContext *s, uint8_t *buf) if ((ret = ff_huff_gen_len_table(s->len[i], s->stats[i], s->vlc_n, 0)) < 0) return ret; - if (ff_huffyuv_generate_bits_table(s->bits[i], s->len[i], s->vlc_n) < 0) { - return -1; - } + ret = ff_huffyuv_generate_bits_table(s->bits[i], s->len[i], s->vlc_n); + if (ret < 0) + return ret; size += store_table(s, s->len[i], buf + size); } -- 2.40.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".