Opened 3 years ago

Last modified 3 years ago

#1277 new defect

DPB size in HM

Reported by: adarsh Owned by:
Priority: minor Milestone:
Component: HM Version: HM-14.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description (last modified by ksuehring)

The following code exists in the HM decoder in TDecTop::xGetNewPicBuffer:

  if ( !bBufferIsAvailable )
  {
    //There is no room for this picture, either because of faulty encoder or dropped NAL. Extend the buffer.
    m_iMaxRefPicNum++;
    rpcPic = new TComPic();
    m_cListPic.pushBack( rpcPic );
  }

It basically allocates memory more than the size indicated by the DPB, which should not be permitted. It is particularly significant in testing bitstream conformance, to ensure that a bitstream does not accidentally exceed the signalled DPB size but HM decoder doesn't catch it.

Change History (5)

comment:1 Changed 3 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 3 years ago by ksuehring

I think there are many of the profile/level limits currently not being tested on both encoder and decoder.

We would appreciate any patches for additional checks.

comment:3 Changed 3 years ago by ksuehring

  • Milestone HM-15.0 deleted

comment:4 Changed 3 years ago by ksuehring

  • Description modified (diff)

comment:5 Changed 3 years ago by ksuehring

I agree that the code should not be necessary anymore after #1304 is fixed. It should be replaced by an assert().

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

  • Adarsh Krishnan Ramasubramonian(Reporter)
  • David Flynn(Subscriber, Always)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Always)
  • Karsten Suehring(Subscriber, Participant, Always)