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 CB5374368B for ; Sat, 19 Nov 2022 03:00:16 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id F046F68BC15; Sat, 19 Nov 2022 05:00:08 +0200 (EET) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05olkn2057.outbound.protection.outlook.com [40.92.89.57]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6E18768BBE3 for ; Sat, 19 Nov 2022 05:00:02 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fzvlQPTcgyxbAY5IBxjEeyPcoP0WNlxtCFQFSVyBckHUKx/uKOqnqe777M9IhmjdvIuWCEB1wPm44g36GpcgJ8gJwflRhbBp4XiZ9RSS6oCPredUlap4FLGm7jA3vqNFp1nnm4yredmgXkRLo2rR39x0hjL6JKLzI2H0yWjm4gMKJ7ZkzADUXQCR+srDbfDb5m6pooIsIKdz4fbxT9A9RkLvAAg1blHsB17h5fpjz0cRDvMCCA1QyeaCJIN1J6TjAdYZP8fbgwBsJ7I5I9HxOdnyP1Qa6NLP3iqG5FcBkI8+9zfPk2BsoXoGYdIczNjpTSDzzRqdoROKMXk2G5n0vA== 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=3hkLOpP6Dosvy5yHQLnSecKdvxUS5p6a3WtYwgeFMj8=; b=l0QnVXyNaxltfUSSPxW73p8ZQQp9OQnXxLKnJ4p1X8DeLng5kM6Jt9/soCZ1FtAkXAGZwpauTa9JDaVDPcWqOOLCUReI07s1Tc7SHCuVzWqiGUG4nzMJnwiqRRfvI0lFHxPckyByPSMwbA1ws9+mb7DpiB7imbSoxbKGb9r2AL09LwMjKrhmWAHbRj3fc/q4rMEjqWx7T6cCk1PJLpejjMStPKEHPeSEJQkIXwGrWgycozZDUfU+vfcvawcdsrgOSWHBhSYrqCrUkIBDl6YZKP1VzqMHB3S2k4XK6BbnYe9Zqwc6X+zU/NTa0htkchd+oHuoruORQKOG3NGSvXSv/A== 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=3hkLOpP6Dosvy5yHQLnSecKdvxUS5p6a3WtYwgeFMj8=; b=bxhoqxP5LvreB+NpPSmoceeZWOigKb8SXDkL/VxXk2JlCpqxYrRhSpaVDPgbSAi9yBgDOHv1njfSoAeIpIZ6CG7pwVYyUGesy+fHRREpNFINp29KntFEfc9rwUQV2xfrZEm+B7efuCoXiwTrnRjvmYMXQLwlcQ97z4mCpx0YO90I4mzTXagVbaFieowRbQhfcTlsoAU3aBFFPZLkrXHGTqlv2teFLM1+Zn6bPvr7UWaJDuCX9kNm7LE9G4GLMDg9m/h2vwQrtBoxiIeXt/45tr9/S/glL2FXiaNJf72gbqpqWtqvI1x9VBrzIorFIv326FB4B6APXjKmXgPRHq4A7w== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by DU2P250MB0127.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:270::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5834.9; Sat, 19 Nov 2022 02:59:52 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5188:7559:45ea:5f52]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5188:7559:45ea:5f52%5]) with mapi id 15.20.5813.018; Sat, 19 Nov 2022 02:59:52 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 19 Nov 2022 03:59:53 +0100 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [KygkbPV1uku9qE/h3anUhk+sBKyxDGQmcF6femb0fCE=] X-ClientProxiedBy: FR2P281CA0044.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:92::18) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <20221119025955.1758795-4-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8P250MB0744:EE_|DU2P250MB0127:EE_ X-MS-Office365-Filtering-Correlation-Id: 88124dfa-72e4-411d-37e3-08dac9da2159 X-MS-Exchange-SLBlob-MailProps: EgT5Wr3QDKzQ+EiUC8SRy4slEJiQUIdMI0E9737jnnKWQEaEcUKBgMucpzRJMQUV5xtyQ8LvyT3m9IzIBPpBdH4XOp0jSNfZEGpeIpazT8v5RG5SuGzikhb6x7O5vHbaLTeuwJRLKeYK5VrhZ6dR3cmD726tNvIOCrOE2tllYs/s8YrEh7MubmyV5C8EvwsQUHgYdVfjGFyFIbcLqgjT4he562hchETPQiTOC/bgFmLwalY6phGKdS9jK9lFH8D2WcBu8G1JzINmlq4AbH5gDdSSJFYn4088sQqQ2i7Qx1ROdA2qv+PNT6ip+6VJsBfifAmYGzEIviLr1mn29qh58tHA7Q5SpDERLw+Dn4aez2P4LsaDk4sYQA+Gz8hYqHGICso86ujozzvT5oNayj5FALsQnOu1opd2AcoXxFnbQDq09Q4JyxOLhsn3KUzwxycrwW+0IIz4Fkk+oXCLLSKkB7RDUepnxvP5b7/1DyFK2axWwpHUMvP1NODzwLNmEhYX4PsypeRM3ZTYE4mJoO/NaaSy5ft5unj4hGebJrnFEfBUl2ZveOe3F6ULyYIfqakzNMDkXObcPF+0slgFaJhQTXkzNaSgTX+ksne9lXrWF/Kru/W1WjpQwwcyIJar6xRiHz3ir/35ZiYLC/enO9HQCwZZdKqsZLU4JCo1tkOCG5yh393zAPiWkP2SH2dDZQp3FW8ZxGdVhKknUIfBeH31Rhs5BTsayy3ogQrXKrH1Sx8= X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: AqWZdIdng8aJxcGGCJ+JrVep04dUekgQ5eoIIFAHsoaV2hm6aZ0rGTHFd16rWz9CCZ1CIV8p4/eD2hv3QQp3lne/mAGSSVcUPdM89TjHc92fZEB5eotpuFxLpIMWjUuU0l1N+W1FcPgmIPVpLc1BALy3S54kRYezhsfNaoe6aYy9xpwT8mEsC/teSXJI/i3u2E29aatXVOCbvYHgJyP61doSD7cjlEGGw2GYYntuotk+SG3LJ31A+E39QQpUV2A7NAaoOPxmrebCHE9NLFsBJeQloGeg1pCiixuWrWwkps0pS4M1Q2KeyRs4LyCYDdisgv/Xn36xTYWHObPfmWSGLIAIupiLhb05tjXB9o4Oj8gpqs7q6ZiU1+C955pp+4K25K+4nXJY7dU7p3MS1rIs46wKAU5/3ccftZfNZFsROGxjOnnsskDmrCj+x/On0dLkzDkzYdPfk9Q+p+gXrGT+lScgtuF2pcJF5VcaC/AO9lVMduvHc3gtMCFmB/QlnRJ7WfdrCa4KIvMW8MhktUc4iq55Rw14zlSbtHbuMK7j8pVgxmREyuLrxCtOi6b+3OnNPh07OONPsNl6/n0OfOr2R6u9J0wVoLSXAcZoIm4Jx1A63ETKPvPJtEnRzFQ7DmDm8jg8pwdOqDR/ZSrAV5hkemQm+mwXZEew13hWa5rsLuU= X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?XPGtgEFjII8uX0YSk6nZhsZZAd61mARDdv7UqdFn0BGZ5AfihhWPVmyNSIZy?= =?us-ascii?Q?6QsLbkpJiZxQwo0vw+3ZoMfiN2sEX1V+lC35X69VZ2UFVB1AegsHGE4NmpVJ?= =?us-ascii?Q?AlZthotFwcbHjJTyaU1y0eNwo3WXowPk+o/gJMvGqxzmnViNHoqnFsusWaPA?= =?us-ascii?Q?2RkDKRulrTekJkeFJr+qqQHn1YWdOch+i6ZUUZ3ikwnCbEHUA1BB45MHj0l8?= =?us-ascii?Q?Nnj8bdD5fpGh/mLE7a7kjyW2V2kgcVyoEDbUVzH3aXkmeOM4RNhjxQX8HE69?= =?us-ascii?Q?og++tZiH1TRZq+GoeVBvNwPlICldzUlsfPLQNZj6zU+OhGTWwtSyr2cCAtO6?= =?us-ascii?Q?vxLfuh5DKh8oQsU+Cxj2qnMIIvYZz+eRiTERkkN2H5XaPFa7lfLMJlN2WIR4?= =?us-ascii?Q?qN2+anCZ24rVyBuDRHHu6R7pTjNOD6PX3+Y2HdB2+q6KmeHGFGeO0jz6IXsY?= =?us-ascii?Q?pRSd3Ds5BTIRKEYY4MJjOE9UGBM47LfXp0dulsxh9dIQz7zH+Y5p+zUCTIXJ?= =?us-ascii?Q?+RfKB+QPAqk0rToBx/LwPq+mrG7IfTVZPuATGTvTuI24L8g96qhKITdlcETx?= =?us-ascii?Q?n1R19qkdwjhvcsFYZHlHucAZTvHxspvYuqOTo5s12fd15JNcfeIRcn+JwVpG?= =?us-ascii?Q?SUzXXQya5PlpMFBv/S+z3km43o0djtWuwfDUuxjHKqkC4qJH4O94g+cLNUQg?= =?us-ascii?Q?ijUrGN4ymnfiC/yraT9PRlAonrTLG+t86IDymA8EtKoMXrgkewkDv0LXcX7q?= =?us-ascii?Q?t3HPJ66cXHvlh6ah3E27Ym/ZZ7eo8uQgUXNVPS0GE7eTZfSYDhRXKU2Jlrly?= =?us-ascii?Q?7LMLzc5c8T3Y+wjC8/aynsVj0/5jD+oMF9/sfMkHYT41jmEwTE7yuzRlQYT/?= =?us-ascii?Q?3G7TqjZzhqmOU6m0l7JUv8YqJ93GORXs7f1O3PWjgEulyM3Q4+Aj4mFKoXqf?= =?us-ascii?Q?4aPji7br9KsNXYojKPWWihSBJWHpSa6cXNlwD8EeF/dXGBM8308WEXvdSogc?= =?us-ascii?Q?fQc51csuqx6QgfFDDFK5etjr/nJTafdUuN0IYTUIGzTYGuVW7/tRqndu6ZwA?= =?us-ascii?Q?ma3KdXcbhG9Vslk/ZmAqZWbteAFX/UJJwOsNv+s4ZNVGhu0zSEVPJcZmu98u?= =?us-ascii?Q?kAllI1TzVO09W66PxQnF29PZ6zw8uxnbZ7NusEfllAbQs4mEG3Gc5XveDCm4?= =?us-ascii?Q?FMKaQlkT268dK0t5Gz2qJbC6F8Vkq6tD+kX9AslMIlaDaweLKVSAlD/+ZNl/?= =?us-ascii?Q?03Htfnmlwy5rM2pACF1wRgdYjiot/LPBvyBazbO52Q41QRMcKxpPdHct8mO4?= =?us-ascii?Q?2t/Q77DbN6WNFju51TXIsxlK?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 88124dfa-72e4-411d-37e3-08dac9da2159 X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2022 02:59:52.8238 (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: DU2P250MB0127 Subject: [FFmpeg-devel] [PATCH 5/7] avcodec/flashsvenc: Change check for first frame 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: AVCodecContext.frame_number is actually only incremented in case encoding was successfull; if e.g. the ff_alloc_packet() below fails, it won't be incremented and therefore it is possible for the previous_frame buffer to be allocated for multiple first frames, leaking every one except the last. So check for whether there already is a previous frame instead. Signed-off-by: Andreas Rheinhardt --- libavcodec/flashsvenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/flashsvenc.c b/libavcodec/flashsvenc.c index 18dfde0e24..f42ef73b37 100644 --- a/libavcodec/flashsvenc.c +++ b/libavcodec/flashsvenc.c @@ -199,7 +199,7 @@ static int flashsv_encode_frame(AVCodecContext *avctx, AVPacket *pkt, int opt_w = 4, opt_h = 4; /* First frame needs to be a keyframe */ - if (avctx->frame_number == 0) { + if (!s->previous_frame) { s->previous_frame = av_mallocz(FFABS(p->linesize[0]) * s->image_height); if (!s->previous_frame) { av_log(avctx, AV_LOG_ERROR, "Memory allocation failed.\n"); -- 2.34.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".