Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#1303 closed defect (fixed)

Next CVS headers are activated before previous has filters executed because of APS SEI

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

Description

Attached stream has two CVS, each CVS has VPS/SPS/PPS ID set to zero which is allowed by spec both because headers are placed between CVS NAL units and because headers have identical information. But when decoder encounters second APS SEI it activates headers deleting old data structures from memory even though SAO filter is not executed for the last CVS frame. In this example PCM restoration fails because SPS->m_bPCMFilterDisableFlag, SPS->m_uiPCMBitDepthLuma and SPS->m_uiPCMBitDepthChroma values have garbage after SPS memory block is freed from memory. Other references to headers, e.g. in deblocking may get garbage too.

Attachments (1)

test.hevc (103.3 KB) - added by gregory 6 years ago.
Example stream with two CVS

Download all attachments as: .zip

Change History (5)

comment:1 Changed 6 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

Changed 6 years ago by gregory

Example stream with two CVS

comment:2 Changed 6 years ago by ksuehring

  • Milestone HM-15.0 deleted

comment:3 Changed 6 years ago by karlsharman

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r4219 by placing a constant instance of the SPS and PPS within
TComPicSym. The SPS/PPS pointers within a slice point to the instance
within the TComPicSym. The decoder will abort if the active PPS/SPS
changes whilst decoding a picture.

comment:4 Changed 6 years ago by karlsharman

  • Milestone set to HM-16.3

Batch modify of recently closed tickets to set milestone to HM-16.3.

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)
  • Gregory Shimansky(Reporter)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)