Opened 13 years ago

Closed 13 years ago

#119 closed defect (fixed)

A bug in chroma deblocking

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

Description

When CU size is 8, m_aapucBS[iDir][0][uiBsAbsIdx] for some values of uiBsAbsIdx are accessed before being calculated.

Attachments (2)

TComLoopFilter.cpp​ (61.6 KB) - added by andreyn 13 years ago.
Bugfix. (To activate, set #define BUGFIX_119 1)
TypeDef.h​ (23.9 KB) - added by andreyn 13 years ago.
TypeDef.h with #define BUGFIX_119 0 for the fix. Switched off.

Download all attachments as: .zip

Change History (7)

Changed 13 years ago by andreyn

Bugfix. (To activate, set #define BUGFIX_119 1)

Changed 13 years ago by andreyn

TypeDef.h with #define BUGFIX_119 0 for the fix. Switched off.

comment:1 Changed 13 years ago by andreyn

The fix makes a separate decision on chroma filtering for each part of the chroma block border (down to 2 chroma pixels in length). Before, the decision was made for the edge of 8 chroma pixels in lengths. Therefore, for CUs of size 8, the algorithm was looking into m_aapucBS[iDir][0][uiBsAbsIdx] for the uiBsAbsIdx, for which the values have not been calculated yet.

comment:2 Changed 13 years ago by fbossen

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

Fixed in r434

comment:3 Changed 12 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:4 Changed 12 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:5 Changed 12 years ago by davidf

  • Cc jct-vc@… added
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

  • Andrey Norkin(Reporter, Participant)
  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)