Ticket #15789 (closed defect: fixed)

Opened 2 months ago

Last modified 2 months ago

[PATCH] Memory leaks in src/common/imagbmp.cpp

Reported by: ettl.martin Owned by:
Priority: normal Milestone:
Component: GUI-all Version: dev-latest
Keywords: memory leak Cc:
Blocked By: Patch: yes
Blocking:

Description

Please review the attached patch. It fixes three memory leaks in src/common/imagbmp.cpp.

Best regards and many thanks

Martin Ettl

Attachments

fix_memleak.patch download (0.6 KB) - added by ettl.martin 2 months ago.
proposed fix

Change History

Changed 2 months ago by ettl.martin

proposed fix

Changed 2 months ago by VZ

  • status changed from new to closed
  • resolution set to fixed

(In [75503]) Fix possible memory leak in BMP reading code.

Don't forget to free the arrays even in case of an error while reading from
the stream.

Closes #15789.

Changed 2 months ago by VZ

(In [75505]) Use wxScopedArray instead of raw new[]/delete[] and fix memory leak.

Don't leak memory in case of error when reading from the stream in BMP loading
code.

Closes #15789.

Note: See TracTickets for help on using tickets.