Opened 5 years ago

Closed 5 years ago

#1056 closed defect (fixed)

heap corruption when decoding bitstream with HRD parameters

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

Description

Caused by incorrect delete:

diff --git a/source/Lib/TLibCommon/TComSlice.cpp b/source/Lib/TLibCommon/TComSli
index f7ca310..8728aa0 100644
--- a/source/Lib/TLibCommon/TComSlice.cpp
+++ b/source/Lib/TLibCommon/TComSlice.cpp
@@ -1203,9 +1203,9 @@ TComVPS::TComVPS()

 TComVPS::~TComVPS()
 {
-  if( m_hrdParameters    != NULL )     delete m_hrdParameters;
-  if( m_hrdOpSetIdx      != NULL )     delete m_hrdOpSetIdx;
-  if( m_cprmsPresentFlag != NULL )     delete m_cprmsPresentFlag;
+  if( m_hrdParameters    != NULL )     delete[] m_hrdParameters;
+  if( m_hrdOpSetIdx      != NULL )     delete[] m_hrdOpSetIdx;
+  if( m_cprmsPresentFlag != NULL )     delete[] m_cprmsPresentFlag;
 }

 // ----------------------------------------------------------------------------

Change History (2)

comment:1 Changed 5 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 5 years ago by fbossen

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

Fixed in r3390

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(Always)
  • Karsten Suehring(Subscriber, Always)