Opened 11 years ago Last modified 10 years ago #1277 new defectDPB size in HM
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 11 years ago by DefaultCC Plugin
comment:2 Changed 11 years ago by ksuehringcomment:3 Changed 11 years ago by ksuehring
comment:4 Changed 10 years ago by ksuehring
comment:5 Changed 10 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
|
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.