From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ffmpeg-devel-bounces@ffmpeg.org>
Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100])
	by master.gitmailbox.com (Postfix) with ESMTPS id E8D944CD7D
	for <ffmpegdev@gitmailbox.com>; Mon, 14 Apr 2025 07:13:35 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8B5F5687C6A;
	Mon, 14 Apr 2025 10:13:31 +0300 (EEST)
Received: from EUR05-DB8-obe.outbound.protection.outlook.com
 (mail-db8eur05on2056.outbound.protection.outlook.com [40.107.20.56])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 90CB56879D6
 for <ffmpeg-devel@ffmpeg.org>; Mon, 14 Apr 2025 10:13:25 +0300 (EEST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;
 b=Di13qWjr+/0Hugen9A/Zc+ewbAkQijFibcrKx+VozeXepy58SSfa99Nu9OZVlXTPBEmTCYPXw8uYHMSERR8tLEEonJm4cBNaTJg/hVsVczC6XVDt/mSF+byelDeG5KDQoD9yMIek1DKVPIlxAHo/PsjLQ+C81uXtnvCuYSF/r+LpRMIQSyfYdZfopDRNASdHaK9dRdaYwweDUjgjyH4y/zr6PyLpyozmKHQVfnhVyqNiBePjdTrKsiI5KzOX3Kaijc6OtCY7owa45x7C0/8o+VkrPCEl5sWVjOcPMgkj5DpP/rDzTiK8VLYXmO5RxRckuRxaUAMYvZ+yGoFkTqODIQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; 
 s=arcselector10001;
 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=s6/icfDOB6awHActUN/xBjSAuRUUxcsFpRnY7j0bDFk=;
 b=kzn8T3PbhqWLeObrNziSOxBHUOIDl2HKJyO5Igqb49YmJVBBV2uz9PSU7KKJ7HaQT25Tmv6CBs8EcBuR6UodwE6mNlj3wrTfwpCgfN+4pHpZcuEDBYIJtWdhnO3JpDBBkl2ZTO7HUKi1Ihmo3qxBgH1Y9Mij88pNqP+H2nfrGdpc1ukkLdMWEOxrIfiyzws2I3ps5ZMRProA1p1TtmRKUNwZMNMwCkeyQ63BlBQlRNhABKVQTuOtYwPdZTQQpkdTPs+N9te3/fk7zjc8hivG58HPqzFaGDAO5O9dZPq3Z5o1QdkTwGrA2t3aGjWlLWhOk8uaohF9HLQFGKdp+Gd+zA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass
 smtp.mailfrom=axis.com; dmarc=pass action=none header.from=axis.com;
 dkim=pass header.d=axis.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=axis.com; s=selector1; 
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=s6/icfDOB6awHActUN/xBjSAuRUUxcsFpRnY7j0bDFk=;
 b=PM0nXzBUtnhXKoZiL70AJcTGV/M4p8WCwkNkW+KiBkhJZR2baGp89umQKL3M+PkOSXNyHJcxP+5jzo6zvQ1O1zseBQOY44/dh+9Ci3qVlfT/e/w1ZJZFzrSy+XjW82wVmXM632zAcfhReamRRiig9ABCELn1vBUz1RaXIPqTmU0=
Authentication-Results: dkim=none (message not signed)
 header.d=none;dmarc=none action=none header.from=axis.com;
Received: from DB9PR02MB8372.eurprd02.prod.outlook.com (2603:10a6:10:39b::10)
 by DBBPR02MB10577.eurprd02.prod.outlook.com (2603:10a6:10:539::15)
 with Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8632.33; Mon, 14 Apr
 2025 07:13:22 +0000
Received: from DB9PR02MB8372.eurprd02.prod.outlook.com
 ([fe80::30ae:44fa:e2ad:3541]) by DB9PR02MB8372.eurprd02.prod.outlook.com
 ([fe80::30ae:44fa:e2ad:3541%3]) with mapi id 15.20.8632.030; Mon, 14 Apr 2025
 07:13:22 +0000
Message-ID: <d61eecdf-a68e-4a60-9e99-0c25e6d54e04@axis.com>
Date: Mon, 14 Apr 2025 09:13:21 +0200
User-Agent: Mozilla Thunderbird
To: ffmpeg-devel@ffmpeg.org
References: <20250413155645.805210-1-realparallelc@gmail.com>
Content-Language: en-US
From: Chris Hodges <Chris.Hodges@axis.com>
In-Reply-To: <20250413155645.805210-1-realparallelc@gmail.com>
X-ClientProxiedBy: MM0P280CA0047.SWEP280.PROD.OUTLOOK.COM
 (2603:10a6:190:b::28) To DB9PR02MB8372.eurprd02.prod.outlook.com
 (2603:10a6:10:39b::10)
MIME-Version: 1.0
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: DB9PR02MB8372:EE_|DBBPR02MB10577:EE_
X-MS-Office365-Filtering-Correlation-Id: 8760edda-786e-4368-4be5-08dd7b23d754
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|1800799024|366016;
X-Microsoft-Antispam-Message-Info: =?utf-8?B?QUMvcHR1eXA4OGx1VTk2b3F6eWw0TVBXMDlzTXFHeENLaTRKbSthSHlZK2tM?=
 =?utf-8?B?Vk1Ld05IWHdNU240Y1ZBN0llRjJ2MFpiZ1hhbUV5K1ZaYlVSZENvcGZ6MmVY?=
 =?utf-8?B?Q2dXZ3Jkb0syRXVlOTNSTmhiRWt0bWUzOWxhWE1ENXZYdldxbWQ1M3gzUzk1?=
 =?utf-8?B?VEVYTEVSbGtYRDM5TlBlZTJTWXpaT2JoSWRxeXpabU9rUVVYOTdsVkpua051?=
 =?utf-8?B?NzNxYnhBNU90eE9DOXFuNCtUbjVWSTBDRXhYb1RkTzRmcXNRU0U1THoza3Z3?=
 =?utf-8?B?Q2NiM3A2bER6OE1jK29rTjdhKzVqM1ZZSDdnTC9ucnMvRGhhMlA5STFuSC8z?=
 =?utf-8?B?amJtOTU5dkd6Z1VBczJxbnZsOU1RblFwS1c0bjFhbEVXeFZpd1BjQzh4Z1VM?=
 =?utf-8?B?am8rNmxyeUQvUEdIajVVMWVvaW53WHdrcGpFYUVyWTA5NEFTMjc0Slk1S3F4?=
 =?utf-8?B?SlpkaDBlaW5UMGlaQzVDYmNNUTliaGt4TTZFa3huSDdGaFdIWUdFRU9tcmtm?=
 =?utf-8?B?N29QMWx6UTFleHo5U0NPV1g3THpRRHdDS3J4RDBkNEprL0tQbm9TcUppOE81?=
 =?utf-8?B?VWZPSW0zL2l5OEwwVjhwampGS1B4NDgrVFRqWm1qaVpEZnZSUEh2bnlsRjRa?=
 =?utf-8?B?STAyWlltdWRQWWhsVVZiL2hEcDNXQTk4YTE1dUxVTFlnU1pQcDd1YTlpV1NY?=
 =?utf-8?B?QXdWeVBTL3c2S3N4MlBzTGxaNjJFNVl4WkJXTzhQT0JtejRvYlowZ3gwdEJY?=
 =?utf-8?B?aU1TeXBhR2Q3dkFGQXk3R0ZTbnd3eFhBUDlhRW5qWDlrTFN3UTFyd3ptemNE?=
 =?utf-8?B?VFp6dDhZZlRIdStvcWNWQ2RjVlNUYm9WK0RPWUNRMG1OR3cweEJJaUR6UTUz?=
 =?utf-8?B?SXdQQXRVS2VhTFZiTlJWSVJuV1F4NDI4TGp0eXRQUjgwRnF6eWQweE10Umth?=
 =?utf-8?B?UUtBWUxCYzhxUUJPRjJOWUk5empGVFl6LzVGNHlTOGMySDNBK05SemxVV3R5?=
 =?utf-8?B?NSt5a0swWXN4NXFBcEYzREUrd3BJMi9EZTlXUkRkRDBmQmZVMmZuRnNnWEZH?=
 =?utf-8?B?YnBTN0Jmbys1NDFBZ2o4SDRIT1lveHUrUHZuYXYvaWlvOWRFLzdJZ1JVdXRS?=
 =?utf-8?B?dkpuNVc4UDZXNHR1aFM3S003dWtvTjljcGtuUXJScXVpVXF0S01hbnhOZ2tp?=
 =?utf-8?B?K2Q4V1BHZ3IrWXFhWE5Wc0hOb3grYXhHdGtpZVp6dmFKeHpCbzJXYWFHaGNq?=
 =?utf-8?B?WDM4SXNHNHVXZkRoZVNEek41bTFsRUJlTUQ0VWNGWTBWNDc5eVpxMVlaL3ZU?=
 =?utf-8?B?THNBTHNQNDBCek0wVThDUHZEUzlqay9SSzZvUGdTRldRbFhTT2dqQ2s4a3Ex?=
 =?utf-8?B?QmdLazJBSXhRYlMxUno1ZU1BZlNkMHJ5TTZDdHhBbUVuOWlqckxqNGErN0dh?=
 =?utf-8?B?aEVnbjFvbUZ3VlJ2Tkl0N2NUZzF1UVJ3LzJDL3pSN1NtMTVSUFNhZ2JQSTQx?=
 =?utf-8?B?SzhDZm1XZHRRUTNuOXZRUGxaaC9vMmlLS1hvaWNTYTVaTXhWMlZiVE1hMDZF?=
 =?utf-8?B?VDUwYkVCQWRjR0tHR0Q5alorS2NtSENwL1N2SW9JSFpPMlRHNFJCblJGbDlV?=
 =?utf-8?B?Z01qRklCV09UbGV4a0ZEMmM4VlBKTkcvb3AwQmk4THYxb1IybG1LT2lTbExu?=
 =?utf-8?B?ODI1M1U0NVpHenoxRlhmaGcxc01WdGtyMlpJdHgveWVEcEdobnRyd05yQlZP?=
 =?utf-8?B?emROUmhrL0FTVmdPWmdXRUY4VXZXQkVITUdHRUExdkx6OVNKMmFKbjRMZ2Ri?=
 =?utf-8?B?Q2w0bnZNRUxva0dMUXFPdUJ4RUdvbm5FWWVQLzRJY1ZjdFlIUUxEbGtpbVNm?=
 =?utf-8?B?VURETFRxbjNXa2UyUTV4WEpERDVTZUtPdktFTXpXRzFXK2c9PQ==?=
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;
 IPV:NLI; SFV:NSPM; H:DB9PR02MB8372.eurprd02.prod.outlook.com; PTR:; CAT:NONE;
 SFS:(13230040)(376014)(1800799024)(366016); DIR:OUT; SFP:1101; 
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SUcyaVQ5TWJaN3Jmb2lKQVo2K2pVNVV5TExMNGRyRWViQlhhZGh1Q1VoekxR?=
 =?utf-8?B?SzhHM1c1N2ovLzdVUHRJR05CYXVnRGtsbmVGR0xSTG5vaERCQVlsOG1sbU4w?=
 =?utf-8?B?Q2pjaHBHcldZWnhMNXdpcnlzb0pMZVpBN095V3hlN24wYWRxcDBFKzZpMmxk?=
 =?utf-8?B?QVQ3SU5tbFZmU29xeXFlNGhHaTdZSlVtb1JXVldXalNLRDdnWi9IcTAyS3VH?=
 =?utf-8?B?MHY3M3d1b2NtdkpUNzNVaVNLUHFuRmJvbVJMeElJOER4RENhSjdRUlFLbGRH?=
 =?utf-8?B?VW9Od090MnVpcnhtOVVCUFU4TEplVDNxSkRQOUdTcmcyQ3JKT1NFZVRqNjhh?=
 =?utf-8?B?amRsUytibXQ4clIwS05Sb0xvM2twUnJYbWVRQ0ZYTGZwVnlhT1hsRzdXcUU0?=
 =?utf-8?B?ZjVaZFJpUU96Mm9lTVlnTEhhTE1pd2VJZFQwMlBraFNxQUYwUGpyNDVVNlBJ?=
 =?utf-8?B?S3FkYzZLOFZ6dGl6OFkxV2pqaUlhdEN1SklXdUZLcFFia3dGVmR0WC9reG90?=
 =?utf-8?B?QnRCUFIxMW1GUWg1Ylo5RXA2Nm9Eb1VVelE5OWNnUlBmQ2JmMGNRRGNhRG1x?=
 =?utf-8?B?ekI2V21LRFJUNGFEYzBmM2h5ZEQ4ZVRaQkRhZUlEekJnSGlCamFTUjRDUEhr?=
 =?utf-8?B?UEZTRzdJM0FIYUd6dkxGZUMvRXZYWXFVWEt2SGY0c0lQU1dlb1A0T3doY3FE?=
 =?utf-8?B?WEpXRkJXRnQwZXlnWW9sQzNwYmVtbDd4TEgxa1ZsQ1hLNzY3YkpHM2h1VUJW?=
 =?utf-8?B?MzhHQ2dSRVlTS1A2RVhjUytJenJabElyVU15czdRNHFNMXBpRkRRVTdGcnBG?=
 =?utf-8?B?RlFCa2ZTU29FSi9NSElNZytYRzVZdjZsS2RTTElSR3hCd3kzNHJmMWxSZXRq?=
 =?utf-8?B?TXBVSHdDcTJIRVVsTUw0VnYzajUzaUZwajRWc3NQa1JhZzZqSGJZNlozNFFI?=
 =?utf-8?B?Y3U1Z2lKUXJ1K1U5WUc5NW90WCs0cTVhRW51MDAvT3pVWTdmT0hUVFlZZXZx?=
 =?utf-8?B?L2h2aG0rK2h3QnNnRkNIL0RUOTRhQjQrd1BReDhydWdlUUNvZzdYcDJNZENj?=
 =?utf-8?B?V25xaSsyUkVTMW55QWJtL29GT3p4R1J1ckZmWDk3WjRWeXU0RVkySk9WbHJP?=
 =?utf-8?B?QzZQdGQ5bmNKNTcrR1JaY0M1eVgwczMrcUg3YjNXQ3l4N1FXVEpwWGlONjlv?=
 =?utf-8?B?SFd0cm00cHVTUHlNN0lSa0lXZXZJZmcvMlRYMk9iQ1pFNjJEeEExUjdzMC84?=
 =?utf-8?B?Sm4ySFc4OGZEa2c5TTZKSitnYkR2L1FoM3VBLy9WSWRsVFdoSFhPTWVFZmxJ?=
 =?utf-8?B?LzNvNy9MMmFYNDdnb1BkNlVoTE5LWEY5QUw3ZnJSYUJyQXJKMjZQckFYeDdi?=
 =?utf-8?B?MnZxMEtlL0NwejFGM1BWZzd2RThNWm1QUFpuZHB5TnBSRDFFQmQzODBTWnJn?=
 =?utf-8?B?S3M4T01sRk92M1dURVE5anpUbWp5NUFPYVM3T3RweUJUbGFvdTMxWVpJcGNM?=
 =?utf-8?B?UzBuOEI3bVc4M05Xdm95cS9YNy8wVTcxT3dXRFBMTDJua3h2bmhCbUdXekN6?=
 =?utf-8?B?aU10ZTFHWk1qUVJmbzdzcVVwank4dTZSU0VrSFhWL2xXZk96OWxMYytNT2hy?=
 =?utf-8?B?ZGVHMUEycUFEWkltaXh3T2xoVWkwQklyaXB3Q3FkWTlpdi8xcStERzF1NGxR?=
 =?utf-8?B?MTJ1R3ZnL04vQk9TV0xQYnpNa2UzRkxiQVpxSHlrRm55Z01VRzQzQXM0WldN?=
 =?utf-8?B?NmxtQ0Q1dk1oczdBczdSbnNhVyt4VGtWcUdMNVJnTkNzWlVJeHVsaGo0WWgy?=
 =?utf-8?B?TlJ2SUhPNk1qMWZtd1lKcjUxR0lkdW9GKy90bVNDWXBOQklnY3FaQkt1UzZy?=
 =?utf-8?B?cVR5YzlhalFVR2V0TUNueXM2R0pDd0RlVFVSUGEwSzVqRW9FK0hSMS9QZlVY?=
 =?utf-8?B?T3V5c3h0NHdFWFJLWG9TakQrampCeWtqQlJyYWpFOVN4TmJWVmt0anpLNlhh?=
 =?utf-8?B?c2h6T1JnbWFQbEhZbmd0SXIrcXFveVlYUlA4cmJSYjNlR0FVeEp2YVkvMnVs?=
 =?utf-8?B?eHZKeGJib0hDY1cxcXZqTWo3Q0dacWE5ZjZnRHdHbWs3TXdQd21LR1pIQk1n?=
 =?utf-8?Q?hfnLaGMAgNoGfihbBtody0NhZ?=
X-OriginatorOrg: axis.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8760edda-786e-4368-4be5-08dd7b23d754
X-MS-Exchange-CrossTenant-AuthSource: DB9PR02MB8372.eurprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Apr 2025 07:13:22.5449 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 78703d3c-b907-432f-b066-88f7af9ca3af
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 59tnRSDX/tMqlLt31S6X0+CUIBZnZ647dvAgKIq4X6MkP7CYMaBJVtg7lJ/ckzT/lKIijAVD1DicG1FSDDsZeA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR02MB10577
Subject: Re: [FFmpeg-devel] [PATCH] avformat/rtpdec_av1: Fix fragment
 continuation check when OBU_HAS_SIZE_FIELD is set
X-BeenThere: ffmpeg-devel@ffmpeg.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org>
List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe>
List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel>
List-Post: <mailto:ffmpeg-devel@ffmpeg.org>
List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help>
List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe>
Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Errors-To: ffmpeg-devel-bounces@ffmpeg.org
Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org>
Archived-At: <https://master.gitmailbox.com/ffmpegdev/d61eecdf-a68e-4a60-9e99-0c25e6d54e04@axis.com/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>

Hi there,

On 4/13/25 17:56, Parallelc wrote:

> When OBU_HAS_SIZE_FIELD is set in the OBU header, frag_obu_size remains 0.
> The code used !frag_obu_size to check for unexpected fragment continuation,
> which resulted in incorrect drops. Introduce expect_frag_cont to explicitly
> track continuation expectation.

Good find! I didn't have sources that kept the OBU size field (just curious,
what did you use to generate these RTP streams?), and your fix looks good
to me (although I didn't verify it).

-- 
Best regards,
Chris

_______________________________________________
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".