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 60291496F1 for ; Tue, 19 Mar 2024 10:18:53 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 716BC68D333; Tue, 19 Mar 2024 12:18:50 +0200 (EET) Received: from sender-op-o18.zoho.eu (sender-op-o18.zoho.eu [136.143.169.18]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6DC2F68D1FA for ; Tue, 19 Mar 2024 12:18:44 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; t=1710843521; cv=none; d=zohomail.eu; s=zohoarc; b=E9oSs5Llvy7kroaETdW0S7VB246oLkFIQjXjp78MWgXRu9nOgMYpJyrHZeA8DS3IuU7dDWnjaK0LQbo5+sTuxzzBjB3mDbc8aeY5l5LENaRMHgshmPWrnhZoQvHPw+15RoXUO28MEaMutsEbEelsV3H767HhXSFL4n+1xZ3hHNk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.eu; s=zohoarc; t=1710843521; h=Content-Type:Content-Transfer-Encoding:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To:Cc; bh=wUeCtJuKZ5hWwzjrQK3EuFhwm3sauYt7pdx9mNxOP/w=; b=AYLT/cDjZqxp4EbJ/mC1wXAiWHr3vKZn8RoHYwObE5wSHhrNjA+oAI5MLMmMBtxR79uak24XdhZ7lJi0QSJYV/jOg6lEeIkNQILZZrx/yHv3nFaSy7WB3lQj5bM24717tgMSSJLnGE5aMyczdGD/n9d8ZRymVTbnPWfUjaoL5vo= ARC-Authentication-Results: i=1; mx.zohomail.eu; dkim=pass header.i=frankplowman.com; spf=pass smtp.mailfrom=post@frankplowman.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1710843521; s=zmail; d=frankplowman.com; i=post@frankplowman.com; h=Message-ID:Date:Date:MIME-Version:Subject:Subject:To:To:References:From:From:In-Reply-To:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To:Cc; bh=wUeCtJuKZ5hWwzjrQK3EuFhwm3sauYt7pdx9mNxOP/w=; b=ZYj/gV74a9vnlhTi9MTBq3bUhlzPwW/k4a9mZSiFc8nUZoYoLWLX2FjUB1jzdvJC EFt48GLnqvnS/U4Pish3j+D1R5luJo4OQBpl1N7tWACJGwxxg1wEbO6Vu9nDD7Pu7ZH Y1qhl6AnBJJqF1acxqaLhqAeaUBknNXD1O81Kvxk= Received: from [192.168.0.136] (78.32.118.75 [78.32.118.75]) by mx.zoho.eu with SMTPS id 1710843519606967.0653496769636; Tue, 19 Mar 2024 11:18:39 +0100 (CET) Message-ID: <822f504c-deb7-4d18-9222-4ea6ab0345d4@frankplowman.com> Date: Tue, 19 Mar 2024 10:18:36 +0000 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Content-Language: en-GB To: ffmpeg-devel@ffmpeg.org References: From: Frank Plowman In-Reply-To: X-ZohoMailClient: External Subject: Re: [FFmpeg-devel] [PATCH 00/14] avcodec/vvcdec: support subpicture 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 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: On 18/03/2024 14:16, Nuo Mi wrote: > see introductions here: https://dashif.org/docs/VVC%20HLS%20overview%20.pdf > > Frank Plowman (1): > avcodec/vvcdec: support rectangular single-slice subpics > > Nuo Mi (13): > avcodec/vvcdec: NoBackwardPredFlag, only check active pictures > avcodec/cbs_h266: fix sh_collocated_from_l0_flag and > sh_collocated_ref_idx infer > avcodec/vvcdec: derive subpic postion for PPS > avcodec/vvcdec: ff_vvc_decode_neighbour, support subpicture > avcodec/vvcdec: misc, rename x_ctb, y_ctb, ctu_x, ctu_y to rx, ry to > avoid misleading > avcodec/vvcdec: refact out deblock_is_boundary > avcodec/vvcdec: deblock, support subpicture > avcodec/vvcdec: refact, movie the lc->sc assignment to task_run_stage > to simplify the code > avcodec/vvcdec: sao, refact out tile_edge arrays > avcodec/vvcdec: sao, support subpicture > avcodec/vvcdec: alf, support subpicture > avcodec/vvcdec: mvs, support subpicture > avcodec/vvcdec: inter prediction, support subpicture > > libavcodec/cbs_h266_syntax_template.c | 37 ++-- > libavcodec/vvc/vvc_ctu.c | 12 +- > libavcodec/vvc/vvc_ctu.h | 6 +- > libavcodec/vvc/vvc_filter.c | 233 +++++++++++++------------- > libavcodec/vvc/vvc_filter.h | 6 +- > libavcodec/vvc/vvc_inter.c | 79 ++++++--- > libavcodec/vvc/vvc_mvs.c | 35 ++-- > libavcodec/vvc/vvc_ps.c | 150 ++++++++++++++--- > libavcodec/vvc/vvc_ps.h | 4 + > libavcodec/vvc/vvc_thread.c | 66 ++------ > 10 files changed, 384 insertions(+), 244 deletions(-) > > -- > 2.25.1 Thank you. When compiled without a sanitizer, all subpicture conformance bitstreams decode correctly, however when compiled with UBSAN/ASAN this set causes a lot of errors for me. With UBSAN, I get errors similar to: libavcodec/vvc/vvc_ctu.c:1553:13: runtime error: member access within misaligned address 0xffffb52dbd6c for type 'av_alias64', which requires 8 byte alignment 0xffffb52dbd6c: note: pointer points here f0 b3 9a 99 ff ff 00 00 00 be 2d b5 ff ff 00 00 c4 a6 fb d5 aa aa 00 00 c0 f8 2d b5 ff ff 00 00 on the following sequences: * SUBPIC_B_HUAWEI * SUBPIC_C_ERICSSON * SUBPIC_E_MediaTek * LMCS_B_Dolby * CodingTools_E_Tencent With ASAN, I get errors similar to: ================================================================= ==67289==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x60300000067a at pc 0xaaaad00d21a0 bp 0xfffffba438f0 sp 0xfffffba43908 READ of size 2 at 0x60300000067a thread T0 #0 0xaaaad00d219c in subpic_tiles /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:374:12 #1 0xaaaad00d1048 in pps_subpic_slice /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:406:5 #2 0xaaaad00cf2cc in pps_single_slice_per_subpic /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:419:13 #3 0xaaaad00ce8e0 in pps_rect_slice /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:464:9 #4 0xaaaad00cd4f8 in pps_slice_map /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:499:9 #5 0xaaaad00caf64 in pps_derive /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:543:11 #6 0xaaaad00cac00 in pps_alloc /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:576:11 #7 0xaaaad00c64f8 in decode_pps /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:598:11 #8 0xaaaad00c4444 in decode_ps /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:630:11 #9 0xaaaad00c3e70 in ff_vvc_decode_frame_ps /home/frank/FFmpeg/libavcodec/vvc/vvc_ps.c:866:11 ... 0x60300000067a is located 0 bytes after 10-byte region [0x603000000670,0x60300000067a) on the following sequences: * SUBPIC_C_ERICSSON * SUBPIC_D_ERICSSON Note SUBPIC_D_ERICSSON has the ASAN error but no UBSAN error. Cheers, Frank _______________________________________________ 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".