From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 469614CAC1 for ; Wed, 29 Oct 2025 15:50:23 +0000 (UTC) Authentication-Results: ffbox; dkim=fail (body hash mismatch (got b'tn2TYEMQPi9xpUy9v9ycKQEsJQBU/hVsWnLr3vyD540=', expected b'1H3PzBTOtswYgYJ7wYvrQdxzqMHKVFYnu6eOMdiDhUM=')) header.d=ffmpeg.org header.i=@ffmpeg.org header.a=rsa-sha256 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1761753018; h=mime-version : to : date : message-id : reply-to : subject : list-id : list-archive : list-archive : list-help : list-owner : list-post : list-subscribe : list-unsubscribe : from : cc : content-type : content-transfer-encoding : from; bh=tn2TYEMQPi9xpUy9v9ycKQEsJQBU/hVsWnLr3vyD540=; b=EKEhYGawKqKmaATSTa0SOyAbuhUXDIaObPKGc+QoBsrpYXgqYn6y/p+lv59Jucm+fZvto Q/+4B/RMxpimLwKUgPMWhw0pyxvsiJM3jEEfRBiAzzHV01yU5jjfDDJcGYsF4niyGsZCW+C d4w6iaFCAvsg30+GIaozGbQIDcOCmG5AwqTi37V4qGZkNU19A09DxDD9QBPFjbqyGEtEEp1 3pboFuSa34zAL360KsX5DiVInvhpbGT98htX/ckts/SfFRosyOQKt7ZRgX4814CLm6qzV/0 iexWoHb3K0jY+VSZB5hRpoPHcev9zC0f39+dGL1/7/1iCnvnSO8TMvd2aghw== Received: from [172.19.0.2] (unknown [172.19.0.2]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 3BF8068F87C; Wed, 29 Oct 2025 17:50:18 +0200 (EET) ARC-Seal: i=1; cv=none; a=rsa-sha256; d=ffmpeg.org; s=arc; t=1761753007; b=dRMr/ZprCSth8SERpXGEHHCO8RXxly09aXysbAXNdNXTH7uKbIRgxbPhabj6suaow7frR oO9S7O2FFEEzLJuSlGIAyn0Qak466SpJ4ljNCOvTM8k6nb1zmrDAtAC/906uVHyYpi2czSG NtW7nQ1dn4Rx0UCnmnbxQ3UF6qf/aa1R+HbAy5nPd2o9hYlU3KO7S6zCAiVUHlqT0Q0+Sai C2I1qmeKittbSaxnP0UAU1l5cSYBq4JqrHKuFuD/9dcrjWFmLFDU7tQ9caeqAAVo2a5fl3V aMLW7+A69KsuSGffvwmdMfLhFpi7kT4AotBiHwjO2Txt35QvZJMbK5dJ4lXA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=ffmpeg.org; s=arc; t=1761753007; h=from : sender : reply-to : subject : date : message-id : to : cc : mime-version : content-type : content-transfer-encoding : content-id : content-description : resent-date : resent-from : resent-sender : resent-to : resent-cc : resent-message-id : in-reply-to : references : list-id : list-help : list-unsubscribe : list-subscribe : list-post : list-owner : list-archive; bh=HnuCjsIUdEeCCyjvI8SBp8A/hRvWXWRdIvxf2ckaBl4=; b=hfPwfi6Qgkt3PPSPEwAg6Sm5ojCmQ9f/bKEL22A4YQhuwmF7NW1fsGCZ6yYgnJmSQCOGb VnsAqb6PjUFRoD/LE6V0ePhO9x/telFr/wgxnudgECN4+4vVk/dQvAwxsITpjJyRVJlibdh tC5lmzpVCgJWDpfG8pGADqA30iY/NhIYjJLi6yLB5LPFwwmqimhN1XOZ1UZ90LFmkFNyG0S nKTvXH2zQLJI28phoD+XFle6yk4kvmqw0UkwzqLAAloC7+sIXRJsuT1t2C8UJhbGwKFFC9u 1gpsYkfFBiFgY0WyQKaA5FXyK5A92pEr9cNjIlVoqxgL6HR5jYnKDq3ls4Ig== ARC-Authentication-Results: i=1; ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none; dmarc=pass header.from=ffmpeg.org policy.dmarc=quarantine Authentication-Results: ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none (Message is not ARC signed); dmarc=pass (Used From Domain Record) header.from=ffmpeg.org policy.dmarc=quarantine DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1761752999; h=content-type : mime-version : content-transfer-encoding : from : to : reply-to : subject : date : from; bh=1H3PzBTOtswYgYJ7wYvrQdxzqMHKVFYnu6eOMdiDhUM=; b=1SEHeOYdyUZRKZwO/mGYUj3BzwAOg2cVxRQ7i6Iw3aQPyOYSTlnAkeDRF+ze2hKbrKasJ AG0lUbmmIy1d1a/MeP50EqJpweOPuzC0NIj9mk33jyOsSrqFbuDIZY6P7BG2kZXvTOEFzph uWScRJsWJ+wpOymCJiwGNzqhRPwa81AVgdUZH6Ors2qYhPigi0xSjhrnYsDTN7QWOJR6ke5 CMJWJ2IVV8FHRiD2h/eJezdUjk+D3UZFYZ/uiYHam7Im6gWQjhnrYoN0uUoFVl3wX0BIVFe o+ez1FzJzpyW2mD0kSuyWBjfbGTNGxVZTdRznViNAG/FPWxytCmCAgl+Yx/w== Received: from 02c22a36bd31 (code.ffmpeg.org [188.245.149.3]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id EEBAE68F7DF for ; Wed, 29 Oct 2025 17:49:58 +0200 (EET) MIME-Version: 1.0 To: ffmpeg-devel@ffmpeg.org Date: Wed, 29 Oct 2025 15:49:58 -0000 Message-ID: <176175299913.81.7245996935995744872@7d278768979e> Message-ID-Hash: 36HY3VDREVCBKKAWVKHJOEVFR7R45LOR X-Message-ID-Hash: 36HY3VDREVCBKKAWVKHJOEVFR7R45LOR X-MailFrom: code@ffmpeg.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; header-match-ffmpeg-devel.ffmpeg.org-0; header-match-ffmpeg-devel.ffmpeg.org-1; header-match-ffmpeg-devel.ffmpeg.org-2; header-match-ffmpeg-devel.ffmpeg.org-3; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list Reply-To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PATCH] avcodec/vc1_parser: stop splitting ENDOFSEQ markers into separate packets (PR #20787) List-Id: FFmpeg development discussions and patches Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: James Almer via ffmpeg-devel Cc: James Almer Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Archived-At: List-Archive: List-Post: PR #20787 opened by James Almer (jamrial) URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20787 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20787.patch Supersedes #20781 >>From fd3d41c721b50962cb29c9c409904981c82bd2cd Mon Sep 17 00:00:00 2001 From: James Almer Date: Wed, 29 Oct 2025 12:17:56 -0300 Subject: [PATCH 1/2] avcodec/vc1dec: look for ENDOFSEQ markers at the end of a packet's payload This is in preparation for the parser no longer splitting them into their own packets. Signed-off-by: James Almer --- libavcodec/vc1dec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 55225baec9..536fdcb147 100644 --- a/libavcodec/vc1dec.c +++ b/libavcodec/vc1dec.c @@ -834,9 +834,11 @@ static int vc1_decode_frame(AVCodecContext *avctx, AVFrame *pict, if(s->avctx->flags & AV_CODEC_FLAG_LOW_DELAY) s->low_delay = 1; + if (buf_size >= 4 && AV_RB32(&buf[buf_size-4]) == VC1_CODE_ENDOFSEQ) + buf_size -= 4; /* no supplementary picture */ - if (buf_size == 0 || (buf_size == 4 && AV_RB32(buf) == VC1_CODE_ENDOFSEQ)) { + if (buf_size == 0) { /* special case for last picture */ if (s->low_delay == 0 && s->next_pic.ptr) { if ((ret = av_frame_ref(pict, s->next_pic.ptr->f)) < 0) -- 2.49.1 >>From 7929ba5d1fd21ecdc9e3829c40f21b9d41b7c8d9 Mon Sep 17 00:00:00 2001 From: James Almer Date: Wed, 29 Oct 2025 12:47:11 -0300 Subject: [PATCH 2/2] avcodec/vc1_parser: stop splitting ENDOFSEQ markers into separate packets The decode API can handle outputting delayed frames without relying on the parser splitting off the ENDOFSEQ marker. Signed-off-by: James Almer --- libavcodec/vc1_parser.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/vc1_parser.c b/libavcodec/vc1_parser.c index 124a7a771f..05094d782f 100644 --- a/libavcodec/vc1_parser.c +++ b/libavcodec/vc1_parser.c @@ -216,7 +216,8 @@ static int vc1_parse(AVCodecParserContext *s, if (!pic_found && (b == (VC1_CODE_FRAME & 0xFF) || b == (VC1_CODE_FIELD & 0xFF))) { pic_found = 1; } - else if (pic_found && b != (VC1_CODE_FIELD & 0xFF) && b != (VC1_CODE_SLICE & 0xFF)) { + else if (pic_found && b != (VC1_CODE_FIELD & 0xFF) && b != (VC1_CODE_SLICE & 0xFF) + && b != (VC1_CODE_ENDOFSEQ & 0xFF)) { next = i - 4; pic_found = b == (VC1_CODE_FRAME & 0xFF); break; -- 2.49.1 _______________________________________________ ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org