Opened 8 years ago

Closed 8 years ago

#1147 closed defect (fixed)

SAO buffer-overrun (and field-coding)

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

Description

SAO has currently a maximum of 4 temporal layers, and the supplied field-coding randomaccess configuration needs 5.

SAO should be updated for more flexibility (eg making m_depthSaoRate a vector). Meanwhile, an assertion should be added to prevent buffer-overruns from being silent.

In the current 12.0-dev branch, add:

assert(depth<4);

after line 3345 in rdoSaoUnitAll (prior to writing to member variable m_depthSaoRate)

Change History (2)

comment:1 Changed 8 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 8 years ago by fbossen

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

Fixed in r3638

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