Opened 5 years ago

Closed 5 years ago

#1394 closed defect (fixed)

m_LocalRPS is not reset between CVSs, resulting in incorrect long term reference picture count

Reported by: jackh Owned by:
Priority: minor Milestone: HM-16.6
Component: HM Version: HM-16.4
Keywords: Cc: ksuehring, davidf, karlsharman, jct-vc@…

Description

In TDecCavlc::parseSliceHeader() (TDecCAVLC.cpp line 1070), the number of long term reference pictures is set in the RPS if sps->getLongTermRefsPresent() == true. However, the number of long term reference pictures is not set to zero if sps->getLongTermRefsPresent() == false; in this case, it is just left at its previous value.

This causes a bug when there is more than one CVS present in the stream. If the first CVS uses an SPS with long_term_ref_pics_present_flag = 1, and the second uses one with long_term_ref_pics_present_flag = 0, the long term reference picture count will not get reset between CVSs, resulting in an error.

Suggested fix is to reset the contents of TComSlice::m_LocalRPS in TComSlice::initSlice().

Change History (3)

comment:1 Changed 5 years ago by DefaultCC Plugin

  • Cc ksuehring davidf karlsharman jct-vc@… added

comment:2 Changed 5 years ago by ksuehring

  • Milestone changed from HM-16.5 to HM-next

comment:3 Changed 5 years ago by ksuehring

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

fixed in r4449

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