Opened 10 years ago Last modified 9 years ago #1335 new defectNoOutputOfPriorPicsFlag can be mishandled by reference decoder
Description
TDecTop::checkNoOutputPriorPics(), called following an IRAP picture for which NoOutputOfPriorPicsFlag==1 (call this picture A), unsets the output flag on all pictures except pictures whose POC is the same as picture A (this value is held in m_lastPOCNoOutputPriorPics).
This can cause pictures to be output in error if a picture held in the DPB from the previous IRAP has the same POC as picture A. In this case the picture will not have its output flag unset and may be output later.
There is no need for this POC check as picture A will not yet have been added to the DPB when checkNoOutputPriorPics() is called, so there is no danger of incorrectly setting its output flag. I therefore suggest that this condition be removed from checkNoOutputPriorPics(). Since the variable m_lastPOCNoOutputPriorPics no longer has any purpose once this check is removed, the variable can also be removed. Change History (9)comment:1 Changed 10 years ago by DefaultCC Plugin
comment:2 Changed 10 years ago by jackhcomment:3 Changed 10 years ago by ksuehring
comment:4 Changed 10 years ago by ksuehring
comment:5 Changed 10 years ago by ksuehring
comment:6 Changed 10 years ago by ksuehring
comment:7 Changed 10 years ago by ksuehring
comment:8 Changed 9 years ago by ksuehring
comment:9 Changed 9 years ago by ksuehring
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
|
Additionally, it is necessary to change the line:
in TAppDecTop::decode() to:
as otherwise mid-picture slices of CRA pictures will also result in calls to checkNoOutputPriorPics().