mbox series

[FFmpeg-devel,v2,0/8] Dolby Vision extension block parsing

Message ID 20240323192440.38264-1-ffmpeg@haasn.xyz
Headers show
Series Dolby Vision extension block parsing | expand

Message

Niklas Haas March 23, 2024, 7:19 p.m. UTC 
Changes since v1:
- Rebased onto master
- Dropped av_dovi_metadata_alloc_ext(), instead statically allocate
  space for 32 extension blocks (AV_DOVI_MAX_EXT_BLOCKS)
- Removed misleading comment on nlq_pivots
- Fix memory leak on ff_dovi_ctx_flush()
- Switch to using ff_refstruct to properly manage ext_block lifetime
- Properly attach parsed extension blocks to the frame side data
- Avoid UB on get_cie_xy
- Validate num_ext_blocks
- Added commit fixing incorrect return code on invalid data
- Removed redundant extra init_get_bits8
- Reorderd fields to avoid unnecessary padding

Comments

Niklas Haas March 28, 2024, 1 p.m. UTC | #1 
On Sat, 23 Mar 2024 20:19:49 +0100 Niklas Haas <ffmpeg@haasn.xyz> wrote:
> Changes since v1:
> - Rebased onto master
> - Dropped av_dovi_metadata_alloc_ext(), instead statically allocate
>   space for 32 extension blocks (AV_DOVI_MAX_EXT_BLOCKS)
> - Removed misleading comment on nlq_pivots
> - Fix memory leak on ff_dovi_ctx_flush()
> - Switch to using ff_refstruct to properly manage ext_block lifetime
> - Properly attach parsed extension blocks to the frame side data
> - Avoid UB on get_cie_xy
> - Validate num_ext_blocks
> - Added commit fixing incorrect return code on invalid data
> - Removed redundant extra init_get_bits8
> - Reorderd fields to avoid unnecessary padding

Ping, any further comments / objections?

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