Opened 9 years ago

Closed 7 years ago

#576 closed defect (fixed)

Decoding CBF flag can use CNU context

Reported by: pieterkapsenberg Owned by:
Priority: minor Milestone: HM-13.0
Component: HM Version: HM-7.0
Keywords: CNU context cbf Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In TDecSbac::parseQtCbf, the CBF flags for chroma can be decoded with a depth and context increment of 3, which get init'd with a CNU value. A proper value should be chosen instead.

Change History (5)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by jkei

Since the context increment depends on the transform depth whose maximum value is 2, I think, the current HM does not seem to show any improper behaviour.

comment:3 Changed 9 years ago by pieterkapsenberg

Actually that is only in the common conditions. If you configure min CU = 64 and min TU 4x4, then depth can be 3. I've tried this and set a breakpoint showing that the increment was 3.

comment:4 Changed 9 years ago by jkei

Yes, you are correct. I think not only HM but also the text should be changed to handle the case of trdepth=3.

comment:5 Changed 7 years ago by fbossen

  • Milestone changed from HM-7.1 to HM-13.0
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r3776

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • Jungsun Kim(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)
  • Pieter Kapsenberg(Reporter, Participant)