Opened 12 years ago

Closed 12 years ago

Last modified 5 years ago

#9459 closed patch (fixed)

EngineManager memory leak

Reported by: SF/tomaz82 Owned by: digitall
Priority: normal Component: --Other--
Version: Keywords:
Cc: Game:

Description

EngineManager is never destroyed causing a very minor memory leak due to the instance never being deleted.

Patch is against latest trunk as of 2012-09-17

Ticket imported from: #3568569. Ticket imported from: patches/1564.

Attachments (1)

main.cpp.patch (269 bytes ) - added by SF/tomaz82 12 years ago.
Patch to destroy EngineManager at the very end of scummvm_main

Download all attachments as: .zip

Change History (4)

by SF/tomaz82, 12 years ago

Attachment: main.cpp.patch added

Patch to destroy EngineManager at the very end of scummvm_main

comment:1 by digitall, 12 years ago

The attached patch is relatively trivial and my reading of the code shows that EngineMan(ager) instance is never destroyed so can leak. fuzzie concurred that this should be fine. Committed as 03d34f1f927cc3ac16f3f500688239f7b2f1fc8a.

comment:2 by digitall, 12 years ago

Owner: set to digitall
Resolution: fixed
Status: newclosed

comment:3 by digitall, 5 years ago

Component: --Other--
Note: See TracTickets for help on using tickets.