New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix memory leak #763

Merged
merged 10 commits into from Sep 16, 2017

Conversation

Projects
None yet
2 participants
@jgj212
Contributor

jgj212 commented Sep 15, 2017

fix memory leak in tim.c
fix memory leak in sixel.c
fix memory leak in palm.c
fix memory leak in pes.c
fix memory leak in pict.c
fix memory leak in png.c
fix memory leak in miff.c
fix memory leak in meta.c
fix memory leak in map.c
fix memory leak in hrz.c
fix memory leak in gif.c
fix memory leak in flif.c
fix memory leak in dps.c
fix memory leak in cut.c

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 15, 2017

Member

@jgj212 With git push --force you can overwrite your changes and then you won't have to open a new pull request when you make changes .

Member

dlemstra commented Sep 15, 2017

@jgj212 With git push --force you can overwrite your changes and then you won't have to open a new pull request when you make changes .

@jgj212

This comment has been minimized.

Show comment
Hide comment
@jgj212

jgj212 Sep 15, 2017

Contributor

@dlemstra can you send me your email, i have some question. My email is ...

Contributor

jgj212 commented Sep 15, 2017

@dlemstra can you send me your email, i have some question. My email is ...

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 15, 2017

Member

And can you also make sure we release tim_data here: https://github.com/ImageMagick/ImageMagick/blob/master/coders/tim.c#L386

I will e-mail you when I get home tonight. My timezone is GMT+1 so you might be asleep then 😄

Member

dlemstra commented Sep 15, 2017

And can you also make sure we release tim_data here: https://github.com/ImageMagick/ImageMagick/blob/master/coders/tim.c#L386

I will e-mail you when I get home tonight. My timezone is GMT+1 so you might be asleep then 😄

@jgj212

This comment has been minimized.

Show comment
Hide comment
@jgj212
Contributor

jgj212 commented Sep 15, 2017

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 15, 2017

Member

@jgj212 Feel free to add that to your changes.

p.s. I might be able to help you on skype if you need some more help with git.

Member

dlemstra commented Sep 15, 2017

@jgj212 Feel free to add that to your changes.

p.s. I might be able to help you on skype if you need some more help with git.

@jgj212

This comment has been minimized.

Show comment
Hide comment
@jgj212

jgj212 Sep 15, 2017

Contributor

@dlemstra i fixed another bug, can i commit it to my fork? If i commit it, this pull request will be ok or not?

Contributor

jgj212 commented Sep 15, 2017

@dlemstra i fixed another bug, can i commit it to my fork? If i commit it, this pull request will be ok or not?

@jgj212

This comment has been minimized.

Show comment
Hide comment
@jgj212

jgj212 Sep 15, 2017

Contributor

@dlemstra my skype id is ...

Contributor

jgj212 commented Sep 15, 2017

@dlemstra my skype id is ...

@jgj212 jgj212 changed the title from fix memory leak in tim.c to fix memory leak Sep 15, 2017

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 15, 2017

Member

I hope you are aware that you are sharing your email publicly here. You can edit your message and remove your email and skype id if you want. I have copied it.

Member

dlemstra commented Sep 15, 2017

I hope you are aware that you are sharing your email publicly here. You can edit your message and remove your email and skype id if you want. I have copied it.

@dlemstra dlemstra merged commit c9c9044 into ImageMagick:master Sep 16, 2017

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 16, 2017

Member

Thanks for contributing 👍 . I just did a rebase in your repository to resolve some merging issues. You might want to do a new fork if you want to send us more pull requests.

Member

dlemstra commented Sep 16, 2017

Thanks for contributing 👍 . I just did a rebase in your repository to resolve some merging issues. You might want to do a new fork if you want to send us more pull requests.

dlemstra added a commit that referenced this pull request Sep 16, 2017

dlemstra added a commit that referenced this pull request Sep 17, 2017

dlemstra added a commit that referenced this pull request Sep 17, 2017

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Oct 10, 2017

tez
ImageMagick: update to 7.0.7.7
2017-10-07  7.0.7-7 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-7, GIT revision 21432:29003eeed:20171007.

2017-10-06  7.0.7-7 Cristy  <quetzlzacatenango@image...>
  * Correct handling of GIF transparency (reference
    ImageMagick/ImageMagick#831).

2017-10-04  7.0.7-6 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-6, GIT revision 21426:0a1cb507b:20171004.

2017-10-03  7.0.7-6 Cristy  <quetzlzacatenango@image...>
  * Reset the magick_list_initialized boolean when needed (reference
    ImageMagick/ImageMagick#826).
2017-10-02  7.0.7-6 Cristy  <quetzlzacatenango@image...>
  * Reset the magick_list_initialized boolean when needed (reference
    ImageMagick/ImageMagick#826).

2017-10-01  7.0.7-5 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-5, GIT revision 21382:3846f9d97:20171001.

2017-09-28  7.0.7-5 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Support URW-base35 fonts.

2017-09-26  7.0.7-5 Glenn Randers-Pehrson <glennrp@image...>
  * Removed "ping_preserve_iCCP=MagickTrue;" statement that was inadvertently
    added to coders/png.c (reference
    http://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32771).

2017-09-23  7.0.7-4 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-4, GIT revision 21265:bdbc14590:20170923.

2017-09-23  7.0.7-4 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    ImageMagick/ImageMagick#763).

2017-09-17  7.0.7-3 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-3, GIT revision 21202:6e6907ac7:20170917.

2017-09-17  7.0.7-3 ADLab of Venustech
  * Fixed numerous memory leaks (reference
    ImageMagick/ImageMagick#763).

2017-09-15  7.0.7-3 Glenn Randers-Pehrson <glennrp@image...>
  * Stop potential leaks in the JNG decoder (reference:
    ImageMagick/ImageMagick#760).
  * Maximum valid hour is 23, not 24, in the PNG tIME chunk, and maximum
    valid minute is 59, not 60.

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Oct 22, 2017

tez
ImageMagick: update to 7.0.7.7
2017-10-07  7.0.7-7 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-7, GIT revision 21432:29003eeed:20171007.

2017-10-06  7.0.7-7 Cristy  <quetzlzacatenango@image...>
  * Correct handling of GIF transparency (reference
    ImageMagick/ImageMagick#831).

2017-10-04  7.0.7-6 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-6, GIT revision 21426:0a1cb507b:20171004.

2017-10-03  7.0.7-6 Cristy  <quetzlzacatenango@image...>
  * Reset the magick_list_initialized boolean when needed (reference
    ImageMagick/ImageMagick#826).
2017-10-02  7.0.7-6 Cristy  <quetzlzacatenango@image...>
  * Reset the magick_list_initialized boolean when needed (reference
    ImageMagick/ImageMagick#826).

2017-10-01  7.0.7-5 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-5, GIT revision 21382:3846f9d97:20171001.

2017-09-28  7.0.7-5 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Support URW-base35 fonts.

2017-09-26  7.0.7-5 Glenn Randers-Pehrson <glennrp@image...>
  * Removed "ping_preserve_iCCP=MagickTrue;" statement that was inadvertently
    added to coders/png.c (reference
    http://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32771).

2017-09-23  7.0.7-4 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-4, GIT revision 21265:bdbc14590:20170923.

2017-09-23  7.0.7-4 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    ImageMagick/ImageMagick#763).

2017-09-17  7.0.7-3 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-3, GIT revision 21202:6e6907ac7:20170917.

2017-09-17  7.0.7-3 ADLab of Venustech
  * Fixed numerous memory leaks (reference
    ImageMagick/ImageMagick#763).

2017-09-15  7.0.7-3 Glenn Randers-Pehrson <glennrp@image...>
  * Stop potential leaks in the JNG decoder (reference:
    ImageMagick/ImageMagick#760).
  * Maximum valid hour is 23, not 24, in the PNG tIME chunk, and maximum
    valid minute is 59, not 60.

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Oct 28, 2017

spz
Pullup ticket #5586 - requested by he
graphics/ImageMagick: security update

Revisions pulled up:
- graphics/ImageMagick/Makefile.common                          1.157
- graphics/ImageMagick/PLIST                                    1.98
- graphics/ImageMagick/distinfo                                 1.173

-------------------------------------------------------------------
   Module Name:	pkgsrc
   Committed By:	tez
   Date:		Tue Oct 10 19:47:50 UTC 2017

   Modified Files:
   	pkgsrc/graphics/ImageMagick: Makefile.common PLIST distinfo

   Log Message:
   ImageMagick: update to 7.0.7.7

   2017-10-07  7.0.7-7 Cristy  <quetzlzacatenango@image...>
     * Release ImageMagick version 7.0.7-7, GIT revision 21432:29003eeed:20171007.

   2017-10-06  7.0.7-7 Cristy  <quetzlzacatenango@image...>
     * Correct handling of GIF transparency (reference
       ImageMagick/ImageMagick#831).

   2017-10-04  7.0.7-6 Cristy  <quetzlzacatenango@image...>
     * Release ImageMagick version 7.0.7-6, GIT revision 21426:0a1cb507b:20171004.

   2017-10-03  7.0.7-6 Cristy  <quetzlzacatenango@image...>
     * Reset the magick_list_initialized boolean when needed (reference
       ImageMagick/ImageMagick#826).
   2017-10-02  7.0.7-6 Cristy  <quetzlzacatenango@image...>
     * Reset the magick_list_initialized boolean when needed (reference
       ImageMagick/ImageMagick#826).

   2017-10-01  7.0.7-5 Cristy  <quetzlzacatenango@image...>
     * Release ImageMagick version 7.0.7-5, GIT revision 21382:3846f9d97:20171001.

   2017-09-28  7.0.7-5 Cristy  <quetzlzacatenango@image...>
     * Fixed numerous memory leaks (reference
       https://github.com/ImageMagick/ImageMagick/issues).
     * Support URW-base35 fonts.

   2017-09-26  7.0.7-5 Glenn Randers-Pehrson <glennrp@image...>
     * Removed "ping_preserve_iCCP=MagickTrue;" statement that was inadvertently
       added to coders/png.c (reference
       http://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t2771).

   2017-09-23  7.0.7-4 Cristy  <quetzlzacatenango@image...>
     * Release ImageMagick version 7.0.7-4, GIT revision 21265:bdbc14590:20170923.

   2017-09-23  7.0.7-4 Cristy  <quetzlzacatenango@image...>
     * Fixed numerous memory leaks (reference
       ImageMagick/ImageMagick#763).

   2017-09-17  7.0.7-3 Cristy  <quetzlzacatenango@image...>
     * Release ImageMagick version 7.0.7-3, GIT revision 21202:6e6907ac7:20170917.

   2017-09-17  7.0.7-3 ADLab of Venustech
     * Fixed numerous memory leaks (reference
       ImageMagick/ImageMagick#763).

   2017-09-15  7.0.7-3 Glenn Randers-Pehrson <glennrp@image...>
     * Stop potential leaks in the JNG decoder (reference:
       ImageMagick/ImageMagick#760).
     * Maximum valid hour is 23, not 24, in the PNG tIME chunk, and maximum
       valid minute is 59, not 60.


   To generate a diff of this commit:
   cvs rdiff -u -r1.156 -r1.157 pkgsrc/graphics/ImageMagick/Makefile.common
   cvs rdiff -u -r1.97 -r1.98 pkgsrc/graphics/ImageMagick/PLIST
   cvs rdiff -u -r1.172 -r1.173 pkgsrc/graphics/ImageMagick/distinfo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment