Opened 9 years ago

Closed 9 years ago

#954 closed defect (fixed)

VPS extension assert

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

Description

The HM VPS parsing code (in TDecCAVLC.cpp) contains an assert to check that vps_extension_flag is equal to 0 and does not parse vps_extension_data_flag, but the specification requires that when vps_extension_data_flag is equal to 1 conforming decoders shall ignore all data that follow the vps_extension_flag.

The attached patch (based on HM-9.1-dev rev 3218) fixes the problem with a solution that matches how sps_extension_flag is handled.

Attachments (1)

vps_extension_fix.patch (663 bytes) - added by jonatan 9 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

Changed 9 years ago by jonatan

comment:2 Changed 9 years ago by fbossen

  • Milestone set to HM-9.2
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r3275

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)
  • Jonatan Samuelsson(Reporter)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)