Opened 4 years ago

Closed 3 years ago

#1227 closed defect (duplicate)

The HM decoder doesn't check the validity of {vps,sps}_num_reorder_pics

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

Description

According to the specification:

The value of vps_max_num_reorder_pics[ i ] shall be in the range of 0 to vps_max_dec_pic_buffering_minus1[ i ], inclusive.
[...]
The value of sps_max_num_reorder_pics[ i ] shall be in the range of 0 to sps_max_dec_pic_buffering_minus1[ i ], inclusive. 

But the HM decoder doesn't check that at all. The attached patch adds assertions for these conditions.

Attachments (1)

num_reorder_pic_assert.patch (1.1 KB) - added by Smarter 4 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 4 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

Changed 4 years ago by Smarter

comment:2 Changed 4 years ago by ksuehring

  • Milestone HM-13.0 deleted

comment:3 Changed 3 years ago by karlsharman

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

Added to #1367. Closed as duplicate.

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, Always)
  • Frank Bossen(Subscriber)
  • Guillaume Martres(Reporter)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Participant, Always)
  • Karsten Suehring(Subscriber, Participant, Always)