small memory leak

Post any defects you find in the released or beta versions of the ImageMagick software here. Include the ImageMagick version, OS, and any command-line required to reproduce the problem. Got a patch for a bug? Post it here.
Post Reply
ogoodman
Posts: 1
Joined: 2016-07-25T23:35:01-07:00
Authentication code: 1151

small memory leak

Post by ogoodman » 2016-07-25T23:42:05-07:00

There is a small memory leak in in the 7.0.2 source download which is still present in the git repo.

It is in coders/dng.c and coders/mpc.c where entry->module is given a new value without first destroying any existing value.

Let me know if you want a pull request for the fix.

Regards,
Oliver Goodman

User avatar
magick
Site Admin
Posts: 11061
Joined: 2003-05-31T11:32:55-07:00

Re: small memory leak

Post by magick » 2016-07-26T02:59:06-07:00

Thanks for the problem report. We can reproduce it and will have a patch to fix it in GIT master branch @ https://github.com/ImageMagick/ImageMagick later today. The patch will be available in the beta releases of ImageMagick @ http://www.imagemagick.org/download/beta/ by sometime tomorrow.

broucaries
Posts: 467
Joined: 2008-12-21T11:51:10-07:00

Re: small memory leak

Post by broucaries » 2016-08-08T03:09:20-07:00

Does it affect imagemagick 6 ?

Pointer to commit ?
Bastien

User avatar
dlemstra
Posts: 1568
Joined: 2013-05-04T15:28:54-07:00
Authentication code: 6789
Contact:

Re: small memory leak

Post by dlemstra » 2016-08-08T11:28:41-07:00

This is an IM7 only issue. The relevant commit is this one: http://git.imagemagick.org/repos/ImageM ... b432811e97
.NET + ImageMagick = Magick.NET https://github.com/dlemstra/Magick.NET, @MagickNET, Donate

Post Reply