Opened 14 years ago

Closed 14 years ago

#139 closed defect (fixed)

Memory leak in Void TComAdaptiveLoopFilter::create

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

Description

Memory leak in TComAdaptiveLoopFilter::create was reported by valgrind. In fact, m_imgY_temp was allocated in TComAdaptiveLoopFilter::create but was never freed.

It is quite easy to fix this bug: Add the following line in bold into Void TComAdaptiveLoopFilter::destroy(), namely

Void TComAdaptiveLoopFilter::destroy()
{

...
destroyMatrix_int(m_imgY_temp);
destroyMatrix_int(m_filterCoeffSym);
...

}

According to valgrind, there are other minor memory leak. But they are not serious.

Change History (4)

comment:1 Changed 14 years ago by fbossen

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

Seems already fixed

comment:2 Changed 13 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:3 Changed 13 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:4 Changed 13 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

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