search

merces / libpe

The PE library used by @merces/pev
http://pev.sf.net
GNU Lesser General Public License v3.0
115 stars 40 forks source link

Fixed memory leak when cleaning up cached data #30

Closed felipensp closed 3 years ago

felipensp commented 3 years ago

I've noticed a memory leak when testing peres tool.

jweyrich commented 3 years ago

Excellent catch Felipe! I'd suggest freeing it right after https://github.com/merces/libpe/blob/master/resources.c#L623 though. Then lines 624 and 625 could be extinguished. What do you think? Thanks!

felipensp commented 3 years ago

Excellent catch Felipe! I'd suggest freeing it right after https://github.com/merces/libpe/blob/master/resources.c#L623 though. Then lines 624 and 625 could be extinguished. What do you think? Thanks!

Yeah, You're right. I will change the modification as you told me.