Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#1430 closed enhancement (fixed)

TComPicYuv.cpp possible memory leak

Reported by: kolya Owned by:
Priority: minor Milestone: HM-16.8
Component: HM Version: HM-16.7
Keywords: possible memory leak Cc: ksuehring, davidf, karlsharman, jct-vc@…

Description

It is possible to set to NULL valid pointer in HM-16.7.
It seems it is better to check if they points to allocated memory.
See the patch attached.

Attachments (1)

TComPicYuv.cpp.patch​ (1.5 KB) - added by kolya 9 years ago.
TComPicYuv patch

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by kolya

TComPicYuv patch

comment:1 Changed 9 years ago by karlsharman

Fixed in r4692. However, note that this does not capture even a fraction of all possible memory leaks if the software is modified: there remain some objects that require destroying, and copy constructors have also not been defined for many objects that contain pointers.

comment:2 Changed 9 years ago by karlsharman

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

comment:3 Changed 9 years ago by ksuehring

  • Milestone set to HM-16.8
  • Version set to HM-16.7
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)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Subscriber, Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • Nikolay Shlyakhov(Reporter)