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

"Ignoring invalid time value" error is not solved #685

Closed
jeroen opened this Issue Aug 19, 2017 · 35 comments

Comments

Projects
None yet
7 participants
@jeroen
Contributor

jeroen commented Aug 19, 2017

Discussion also here. This bug still exists in 7.0.6.7 and 6.9.9-7.

This bug appears only at certain time of day! Using OS-X Sierra. Currently it is 2.30pm Amsterdam time and the bug does not appear. However if I set the time on my clock to 10.30pm, I immediately get this error when read + writing a png file.

Perhaps the problem is triggered when the UTC date is different from the local date?

Probably introduced here or here. @glennrp can you have a look?

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Aug 19, 2017

Contributor
Contributor

glennrp commented Aug 19, 2017

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Aug 19, 2017

Contributor

It's a problem with the offset: 2017-08-19T09:31:15-04:00
IM is looking for :04:00 instead of -04:00
I have to contend with military timezones (offset Z) too.
I'll post a fix later today.

Contributor

glennrp commented Aug 19, 2017

It's a problem with the offset: 2017-08-19T09:31:15-04:00
IM is looking for :04:00 instead of -04:00
I have to contend with military timezones (offset Z) too.
I'll post a fix later today.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Aug 19, 2017

Contributor

Having some trouble debugging. It seems that setting TZ on Ubuntu does not have the desired effect:

for TIMEZONE in UTC AMS PDT EDT IST
do
TZ=$TIMEZONE export TZ
date
done
Sat Aug 19 14:34:02 UTC 2017
Sat Aug 19 14:34:02 AMS 2017
Sat Aug 19 14:34:02 PDT 2017
Sat Aug 19 14:34:02 EDT 2017
Sat Aug 19 14:34:02 IST 2017

Note that "date" reports the desired timezone but doesn't actually convert the time.

Contributor

glennrp commented Aug 19, 2017

Having some trouble debugging. It seems that setting TZ on Ubuntu does not have the desired effect:

for TIMEZONE in UTC AMS PDT EDT IST
do
TZ=$TIMEZONE export TZ
date
done
Sat Aug 19 14:34:02 UTC 2017
Sat Aug 19 14:34:02 AMS 2017
Sat Aug 19 14:34:02 PDT 2017
Sat Aug 19 14:34:02 EDT 2017
Sat Aug 19 14:34:02 IST 2017

Note that "date" reports the desired timezone but doesn't actually convert the time.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Aug 19, 2017

Contributor

OK never mind that. I forgot that TZ needs offsets. If I write
for TIMEZONE in UTC0 AMS0 PDT8 EDT4 IST18.5
it works:

Sat Aug 19 15:01:12 UTC 2017
Sat Aug 19 15:01:12 AMS 2017
Sat Aug 19 07:01:12 PDT 2017
Sat Aug 19 11:01:12 EDT 2017
Fri Aug 18 20:31:12 IST 2017

Contributor

glennrp commented Aug 19, 2017

OK never mind that. I forgot that TZ needs offsets. If I write
for TIMEZONE in UTC0 AMS0 PDT8 EDT4 IST18.5
it works:

Sat Aug 19 15:01:12 UTC 2017
Sat Aug 19 15:01:12 AMS 2017
Sat Aug 19 07:01:12 PDT 2017
Sat Aug 19 11:01:12 EDT 2017
Fri Aug 18 20:31:12 IST 2017

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Aug 19, 2017

Contributor

The main problem was the use of unsigned int for "addhours" which is
negative east of the zero timezone.

Fixed in IM7: commit 295c896
IM6: commit 1f54a12

There still may remain a bug at the end of short months; I'll clean that up as time permits.

Contributor

glennrp commented Aug 19, 2017

The main problem was the use of unsigned int for "addhours" which is
negative east of the zero timezone.

Fixed in IM7: commit 295c896
IM6: commit 1f54a12

There still may remain a bug at the end of short months; I'll clean that up as time permits.

@jeroen

This comment has been minimized.

Show comment
Hide comment
@jeroen

jeroen Aug 19, 2017

Contributor

Thank you for the quick fix! Hopefully this will make it into the next release of IM6 soon :)

Contributor

jeroen commented Aug 19, 2017

Thank you for the quick fix! Hopefully this will make it into the next release of IM6 soon :)

@jeroen

This comment has been minimized.

Show comment
Hide comment
@jeroen

jeroen Aug 20, 2017

Contributor

I think your commit on the IM6 branch 1f54a12 may have unintentionally reverted 6519c46 ?

Contributor

jeroen commented Aug 20, 2017

I think your commit on the IM6 branch 1f54a12 may have unintentionally reverted 6519c46 ?

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Aug 20, 2017

Contributor
Contributor

glennrp commented Aug 20, 2017

@jeroen jeroen closed this Aug 22, 2017

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Sep 9, 2017

Update to 6.9.9.11
Changelog:
2017-09-03  6.9.9-11 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-11, GIT revision 11969:a12fbb873:20170903.

2017-08-28  6.9.9-11 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Don't overwrite symbolic links when the shred policy is enabled.

2017-08-27  6.9.9-10 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-10, GIT revision 11936:a8112a821:20170827.

2017-08-26  6.9.9-10 Dirk Lemstra <dirk@lem.....org>
  * Fixed thread safety issue inside the pango and librsvg decoder
    (reference: dlemstra/Magick.NET#91).

2017-08-20  6.9.9-9 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-9, GIT revision 11915:5205bda17:20170820.

2017-08-18  6.6.9-9 Glenn Randers-Pehrson <glennrp@image...>
  * Fixed bug with writing tIME chunk when timezone has a negative offset
    (reference: ImageMagick/ImageMagick#685).

2017-08-18  6.9.9-8 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-8, GIT revision 11906:26078285f:20170818.

2017-08-18  6.9.9-8 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-08-12  6.9.9-7 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-7, GIT revision 11893:8c4c56a0e:20170812
    (Windows binaries out of sync).

2017-08-10  6.9.9-6 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-6, GIT revision 11886:af2b102db:20170810.

2017-08-10  6.9.9-6 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-08-10  6.9.9-6 Glenn Randers-Pehrson <glennrp@image...>
  * tests/validate.c: Show the reason for failures in the test logs,
    if available.

2017-08-03  6.9.9-6 Glenn Randers-Pehrson <glennrp@image...>
  * Put UTC time in the PNG tIME chunk instead of local time (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32447).

2017-08-02  6.9.9-5 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-5, GIT revision 11858:7a555e53f:20170802.

2017-08-01  6.9.9-5 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-07-29  6.9.9-5 Glenn Randers-Pehrson <glennrp@image...>
  * Properly set image->colorspace in the PNG decoder (previously
    it was setting image->gamma, but only setting image->colorspace
    for grayscale and gray-alpha images.  Reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32418).
  * Fix improper use of NULL in the JNG decoder (CVE-2017-11750, Reference
    ImageMagick/ImageMagick#632).

2017-07-29  6.9.9-5 Cristy  <quetzlzacatenango@image...>
  * Off by one error for gradient coder (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32416).

2017-07-28  6.9.9-4 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-4, GIT revision 11833:4e81160d6:20170728.

2017-07-25  6.9.9-4 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * coders/png.c: Initialized quantum_scanline to prevent a bad free
    (reference ImageMagick/ImageMagick#621).

2017-07-24  6.9.9-4 Glenn Randers-Pehrson <glennrp@image...>
  * Removed write_chunk_from_profile() from coders/png.c because it
    has not worked at least since version 6.7.6.
  * Removed many redundant checks before RelinquishMagickMemory(),
    which is safe to call with a NULL argument.
  * Removed vpAg chunk write support (we are now writing caNv instead).
  * coders/png.c: Initialized quantum_info to prevent memory leakage
    (reference ImageMagick/ImageMagick#582,
    CVE-2017-11539).
  * coders/png.c: fixed NULL dereference when trying to write an empty MNG
    (CVE-2017-11522, reference
    ImageMagick/ImageMagick#586).

2017-07-24  6.9.9-3 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-3, GIT revision 11809:2bd88257b:20170724.

2017-07-23  6.9.9-3 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-07-23  6.9.9-3 Glenn Randers-Pehrson <glennrp@image...>
  * Fix memory leaks when reading a malformed JNG image:
    ImageMagick/ImageMagick#600).
    ImageMagick/ImageMagick#602).

2017-07-22  6.9.9-2 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-2, GIT revision 11786:21b23bf09:20170722.

2017-07-22  6.9.9-2 Cristy  <quetzlzacatenango@image...>
  * composite -dissolve works again reference
    ImageMagick/ImageMagick#597).

2017-07-21  6.9.9-1 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-1, GIT revision 11782:75f7e994e:20170721.

2017-07-19  6.9.9-1 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-07-15  6.9.9-1 Glenn Randers-Pehrson <glennrp@image...>
  * Don't write a hex-encoded Exif profile when writing the eXIf chunk.
  * Added caNv, eXIf, and pHYs to the list of PNG chunks to be removed
    by the "-strip" option.

2017-07-15  6.9.9-0 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 6.9.9-0, GIT revision 11738:8903861b2:20170715.

2017-07-13  6.9.9-0 Glenn Randers-Pehrson <glennrp@image...>
  * Implemented PNG eXIf chunk support.

2017-07-08  6.9.9-0 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Don't use variable float_t / double_t, bump SO (reference
    ImageMagick/ImageMagick#510).
  * Support DNG images with libraw delegate library.
@sf-butdim

This comment has been minimized.

Show comment
Hide comment
@sf-butdim

sf-butdim Sep 11, 2017

Hi folks, this bug is still present on Mac OS X Sierra v10.12.5 with ImageMagick v7.0.7 (this is the latest version available via homebrew).

Yesterday somwhen between 23:00 and 23:30 I was getting the same error which spontaneously gone around 23:30.

Can you please investigate? This is VERY annoying.

Many thanks in advance.

sf-butdim commented Sep 11, 2017

Hi folks, this bug is still present on Mac OS X Sierra v10.12.5 with ImageMagick v7.0.7 (this is the latest version available via homebrew).

Yesterday somwhen between 23:00 and 23:30 I was getting the same error which spontaneously gone around 23:30.

Can you please investigate? This is VERY annoying.

Many thanks in advance.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 11, 2017

Contributor

Please run with "-debug coder" and show us the line that begins "Date format ..."

Contributor

glennrp commented Sep 11, 2017

Please run with "-debug coder" and show us the line that begins "Date format ..."

@sf-butdim

This comment has been minimized.

Show comment
Hide comment
@sf-butdim

sf-butdim Sep 11, 2017

@glennrp do you mean I need to run it now or when the problem occurs again? Because this happens quite randomly as you know. Please confirm.

sf-butdim commented Sep 11, 2017

@glennrp do you mean I need to run it now or when the problem occurs again? Because this happens quite randomly as you know. Please confirm.

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 11, 2017

Member

You might be able to force the problem again by changing the time on your machine. If you can please post the output as requested by @glennrp.

Member

dlemstra commented Sep 11, 2017

You might be able to force the problem again by changing the time on your machine. If you can please post the output as requested by @glennrp.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 11, 2017

Contributor

Just run it now so I can see what your timestamp format looks like. Run it again when you observe the problem.

Contributor

glennrp commented Sep 11, 2017

Just run it now so I can see what your timestamp format looks like. Run it again when you observe the problem.

@sf-butdim

This comment has been minimized.

Show comment
Hide comment
@sf-butdim

sf-butdim Sep 11, 2017

@dlemstra thank you for quick tipp, I'll give it a try.

@glennrp I'll check this tonight from my private laptop (currently I'm on working one).

I'll update this issue tonight, thank you for help guys.

sf-butdim commented Sep 11, 2017

@dlemstra thank you for quick tipp, I'll give it a try.

@glennrp I'll check this tonight from my private laptop (currently I'm on working one).

I'll update this issue tonight, thank you for help guys.

@sf-butdim

This comment has been minimized.

Show comment
Hide comment
@sf-butdim

sf-butdim Sep 11, 2017

Good evening folks, it just happened to me again between 22:50 and 23:10. Unfortunately I was only able to get the debug stack trace now, when it's back to normal and working. I hope this helps, if not please ping me and I'll try to reproduce the problem tomorrow evening.

convert -debug Coder img1.png img2.png
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4202/Coder
  Enter ReadPNGImage()
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2237/Coder
    Enter ReadOnePNGImage()
    IM version     = 7.0.7-0
    Libpng version = 1.6.32
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2250/Coder
      Zlib version   = 1.2.8
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2254/Coder
        running with   1.2.5
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2282/Coder
      Before reading:
      image->alpha_trait=0
      image->rendering_intent=2
      image->colorspace=23
      image->gamma=0.454545
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2585/Coder
      PNG width: 1280, height: 4327
    PNG color_type: 2, bit_depth: 8
    PNG compression_method: 0
    PNG interlace_method: 0, filter_method: 0
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2855/Coder
      Setting sRGB as if in input
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3175/Coder
      image->gamma=0.454545
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3201/Coder
      image->colorspace=23
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3355/Coder
      Reading PNG IDAT chunk(s)
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3379/Coder
      Converting PNG pixels to pixel packets
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3453/Coder
      Looking for cheap transparent pixel
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3696/Coder
      No transparent pixel was found
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3721/Coder
    image->storage_class=1
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/4155/Coder
    exit ReadOnePNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4287/Coder
    page.w: 0, page.h: 0,page.x: 0, page.y: 0.
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4291/Coder
    image->colorspace: 23
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4296/Coder
  exit ReadPNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WritePNGImage/11982/Coder
  Enter WritePNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WritePNGImage/12574/Coder
    Chunks to be excluded from the output png:
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8395/Coder
    Enter WriteOnePNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8430/Coder
      IM version     = 7.0.7-0
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8432/Coder
      Libpng version = 1.6.32
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8439/Coder
      Zlib version   = 1.2.8
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8443/Coder
        running with   1.2.5
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8611/Coder
      image->storage_class=DirectClass
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8616/Coder
      image->taint=MagickFalse
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8619/Coder
      image->gamma=0.454545
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8870/Coder
      Enter BUILD_PALETTE:
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8875/Coder
        image->columns=1280
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8877/Coder
        image->rows=4327
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8879/Coder
        image->alpha_trait=0
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8881/Coder
        image->depth=8
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8917/Coder
        image->colors=0
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8921/Coder
          (zero means unknown)
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8925/Coder
        Regenerate the colormap
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9039/Coder
        Check colormap for background (65535,65535,65535)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9065/Coder
        No room in the colormap to add background color
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9074/Coder
        image has more than 256 colors
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9268/Coder
        image->colors=0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9292/Coder
        number_transparent     = 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9306/Coder
        number_opaque          > 256
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9310/Coder
        number_semitransparent = 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9327/Coder
        At least one pixel or the background is non-gray
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9330/Coder
      Exit BUILD_PALETTE:
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9863/Coder
      width=1280
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9865/Coder
      height=4327
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9867/Coder
      image_matte=0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9869/Coder
      image->depth=8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9871/Coder
      Tentative ping_bit_depth=8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9956/Coder
      Setting up bKGD chunk (1)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9958/Coder
        background_color index is 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9962/Coder
      ping_bit_depth=8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10100/Coder
    Selecting PNG colortype:
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10158/Coder
      Selected PNG colortype=2
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10195/Coder
      Number of colors: 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10198/Coder
      Tentative PNG bit depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10210/Coder
      Tentative PNG color type: RGB (2)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10215/Coder
      image_info->type: 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10218/Coder
      image_depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10221/Coder
      image->depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10225/Coder
      ping_bit_depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10629/Coder
      PNG color type: RGB (2)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10637/Coder
    Setting up deflate compression
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10640/Coder
      Compression buffer size: 32768
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10647/Coder
      Compression mem level: 9
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10726/Coder
      Compression level:    7
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10735/Coder
    Setting up filtering
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10739/Coder
      Base filter method: ADAPTIVE
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10874/Coder
    Writing PNG header chunks
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11012/Coder
    Setting up gAMA chunk
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11040/Coder
    Setting up cHRM chunk
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11056/Coder
      Setting up bKGD chunk
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11058/Coder
        background color = (255,255,255)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11063/Coder
        index = 0, gray=255
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11258/Coder
    Writing PNG image data
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11261/Coder
      Allocating 3840 bytes of memory for pixels
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11465/Coder
      Writing row of pixels (3)
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11562/Coder
    Wrote PNG image data
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11565/Coder
      Width: 1280
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11568/Coder
      Height: 4327
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11577/Coder
      PNG bit-depth written: 8
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11586/Coder
      PNG color-type written: 2
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11589/Coder
      PNG Interlace method: 0
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11651/Coder
    Setting up text chunk
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11654/Coder
      keyword: 'date:create'
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11651/Coder
    Setting up text chunk
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11654/Coder
      keyword: 'date:modify'
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11727/Coder
    Writing PNG end info
2017-09-11T23:25:37+02:00 0:00.630 0.610u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11795/Coder
    exit WriteOnePNGImage()
2017-09-11T23:25:37+02:00 0:00.630 0.620u 7.0.7 Coder convert[4906]: png.c/WritePNGImage/12639/Coder
  exit WritePNGImage()

sf-butdim commented Sep 11, 2017

Good evening folks, it just happened to me again between 22:50 and 23:10. Unfortunately I was only able to get the debug stack trace now, when it's back to normal and working. I hope this helps, if not please ping me and I'll try to reproduce the problem tomorrow evening.

convert -debug Coder img1.png img2.png
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4202/Coder
  Enter ReadPNGImage()
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2237/Coder
    Enter ReadOnePNGImage()
    IM version     = 7.0.7-0
    Libpng version = 1.6.32
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2250/Coder
      Zlib version   = 1.2.8
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2254/Coder
        running with   1.2.5
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2282/Coder
      Before reading:
      image->alpha_trait=0
      image->rendering_intent=2
      image->colorspace=23
      image->gamma=0.454545
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2585/Coder
      PNG width: 1280, height: 4327
    PNG color_type: 2, bit_depth: 8
    PNG compression_method: 0
    PNG interlace_method: 0, filter_method: 0
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/2855/Coder
      Setting sRGB as if in input
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3175/Coder
      image->gamma=0.454545
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3201/Coder
      image->colorspace=23
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3355/Coder
      Reading PNG IDAT chunk(s)
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3379/Coder
      Converting PNG pixels to pixel packets
2017-09-11T23:25:36+02:00 0:00.010 0.000u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3453/Coder
      Looking for cheap transparent pixel
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3696/Coder
      No transparent pixel was found
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/3721/Coder
    image->storage_class=1
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadOnePNGImage/4155/Coder
    exit ReadOnePNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4287/Coder
    page.w: 0, page.h: 0,page.x: 0, page.y: 0.
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4291/Coder
    image->colorspace: 23
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/ReadPNGImage/4296/Coder
  exit ReadPNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WritePNGImage/11982/Coder
  Enter WritePNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WritePNGImage/12574/Coder
    Chunks to be excluded from the output png:
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8395/Coder
    Enter WriteOnePNGImage()
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8430/Coder
      IM version     = 7.0.7-0
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8432/Coder
      Libpng version = 1.6.32
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8439/Coder
      Zlib version   = 1.2.8
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8443/Coder
        running with   1.2.5
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8611/Coder
      image->storage_class=DirectClass
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8616/Coder
      image->taint=MagickFalse
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8619/Coder
      image->gamma=0.454545
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8870/Coder
      Enter BUILD_PALETTE:
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8875/Coder
        image->columns=1280
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8877/Coder
        image->rows=4327
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8879/Coder
        image->alpha_trait=0
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8881/Coder
        image->depth=8
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8917/Coder
        image->colors=0
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8921/Coder
          (zero means unknown)
2017-09-11T23:25:36+02:00 0:00.130 0.110u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/8925/Coder
        Regenerate the colormap
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9039/Coder
        Check colormap for background (65535,65535,65535)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9065/Coder
        No room in the colormap to add background color
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9074/Coder
        image has more than 256 colors
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9268/Coder
        image->colors=0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9292/Coder
        number_transparent     = 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9306/Coder
        number_opaque          > 256
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9310/Coder
        number_semitransparent = 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9327/Coder
        At least one pixel or the background is non-gray
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9330/Coder
      Exit BUILD_PALETTE:
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9863/Coder
      width=1280
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9865/Coder
      height=4327
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9867/Coder
      image_matte=0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9869/Coder
      image->depth=8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9871/Coder
      Tentative ping_bit_depth=8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9956/Coder
      Setting up bKGD chunk (1)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9958/Coder
        background_color index is 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/9962/Coder
      ping_bit_depth=8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10100/Coder
    Selecting PNG colortype:
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10158/Coder
      Selected PNG colortype=2
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10195/Coder
      Number of colors: 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10198/Coder
      Tentative PNG bit depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10210/Coder
      Tentative PNG color type: RGB (2)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10215/Coder
      image_info->type: 0
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10218/Coder
      image_depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10221/Coder
      image->depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10225/Coder
      ping_bit_depth: 8
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10629/Coder
      PNG color type: RGB (2)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10637/Coder
    Setting up deflate compression
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10640/Coder
      Compression buffer size: 32768
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10647/Coder
      Compression mem level: 9
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10726/Coder
      Compression level:    7
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10735/Coder
    Setting up filtering
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10739/Coder
      Base filter method: ADAPTIVE
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/10874/Coder
    Writing PNG header chunks
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11012/Coder
    Setting up gAMA chunk
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11040/Coder
    Setting up cHRM chunk
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11056/Coder
      Setting up bKGD chunk
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11058/Coder
        background color = (255,255,255)
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11063/Coder
        index = 0, gray=255
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11258/Coder
    Writing PNG image data
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11261/Coder
      Allocating 3840 bytes of memory for pixels
2017-09-11T23:25:36+02:00 0:00.170 0.160u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11465/Coder
      Writing row of pixels (3)
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11562/Coder
    Wrote PNG image data
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11565/Coder
      Width: 1280
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11568/Coder
      Height: 4327
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11577/Coder
      PNG bit-depth written: 8
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11586/Coder
      PNG color-type written: 2
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11589/Coder
      PNG Interlace method: 0
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11651/Coder
    Setting up text chunk
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11654/Coder
      keyword: 'date:create'
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11651/Coder
    Setting up text chunk
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11654/Coder
      keyword: 'date:modify'
2017-09-11T23:25:37+02:00 0:00.620 0.600u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11727/Coder
    Writing PNG end info
2017-09-11T23:25:37+02:00 0:00.630 0.610u 7.0.7 Coder convert[4906]: png.c/WriteOnePNGImage/11795/Coder
    exit WriteOnePNGImage()
2017-09-11T23:25:37+02:00 0:00.630 0.620u 7.0.7 Coder convert[4906]: png.c/WritePNGImage/12639/Coder
  exit WritePNGImage()
@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 12, 2017

Contributor

The problem seems to be due to use of unsigned arithmetic in the timezone calculations. Fixed in IM7 commit 43f1a50 and IM6 commit bdb8121

Contributor

glennrp commented Sep 12, 2017

The problem seems to be due to use of unsigned arithmetic in the timezone calculations. Fixed in IM7 commit 43f1a50 and IM6 commit bdb8121

@dlemstra dlemstra reopened this Sep 12, 2017

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 12, 2017

Member

@glennrp It appears this issue is still not resolved. You can force a failure with the following commands:

convert logo: -define date:modify=2017-09-10T20:35:00+03:30 test.png
convert logo: -define date:modify=2017-09-10T15:30:00+03:30 test.png
Member

dlemstra commented Sep 12, 2017

@glennrp It appears this issue is still not resolved. You can force a failure with the following commands:

convert logo: -define date:modify=2017-09-10T20:35:00+03:30 test.png
convert logo: -define date:modify=2017-09-10T15:30:00+03:30 test.png
@sf-butdim

This comment has been minimized.

Show comment
Hide comment
@sf-butdim

sf-butdim Sep 12, 2017

@glennrp thank you for an update on that. When could we count this to be fixed? In the next released version?

sf-butdim commented Sep 12, 2017

@glennrp thank you for an update on that. When could we count this to be fixed? In the next released version?

@ferencbeutel4711

This comment has been minimized.

Show comment
Hide comment
@ferencbeutel4711

ferencbeutel4711 Sep 12, 2017

at 22:00 GMT+2 this happened for me aswell today
edit: lasted exactly until 23:00 GMT+2, so 1 hour. also, this happens every day.

ferencbeutel4711 commented Sep 12, 2017

at 22:00 GMT+2 this happened for me aswell today
edit: lasted exactly until 23:00 GMT+2, so 1 hour. also, this happens every day.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 12, 2017

Contributor

Your test is not failing for me with the GIT heads for im6 and im7. However, test.png ends
up with a tIME chunk containing the current time not the defined time.

Contributor

glennrp commented Sep 12, 2017

Your test is not failing for me with the GIT heads for im6 and im7. However, test.png ends
up with a tIME chunk containing the current time not the defined time.

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 12, 2017

Member

I just tested with the latest git head of IM7 and I get the following output:

I:\>convert logo: -define date:modify=2017-09-10T20:35:00+03:30 test.png
convert: Ignoring invalid time value `test.png' @ warning/png.c/MagickPNGWarningHandler/1744.

I think these are incorrect:

  if (minute > 60)
  {
     hour++;
     minute-=60;
  }
  if (hour > 24)
  {
     day ++;
     hour -=24;
  }

This means that hour == 24 and minute == 60 are allowed.

Member

dlemstra commented Sep 12, 2017

I just tested with the latest git head of IM7 and I get the following output:

I:\>convert logo: -define date:modify=2017-09-10T20:35:00+03:30 test.png
convert: Ignoring invalid time value `test.png' @ warning/png.c/MagickPNGWarningHandler/1744.

I think these are incorrect:

  if (minute > 60)
  {
     hour++;
     minute-=60;
  }
  if (hour > 24)
  {
     day ++;
     hour -=24;
  }

This means that hour == 24 and minute == 60 are allowed.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 12, 2017

Contributor

Apparently I can't test in timezones other than my own. I set the environment variable TZ but it seems to have little effect (It changes the format of the output of "date" but not the time).

Contributor

glennrp commented Sep 12, 2017

Apparently I can't test in timezones other than my own. I set the environment variable TZ but it seems to have little effect (It changes the format of the output of "date" but not the time).

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 12, 2017

Contributor

If you run with -debug coder, what does the "png_set_tIME" line look like?

Contributor

glennrp commented Sep 12, 2017

If you run with -debug coder, what does the "png_set_tIME" line look like?

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Sep 12, 2017

Member

I cannot reproduce it on the command line anymore. But I can force the exception by changing the code to this:

      else
        {
          (void) LogMagickEvent(CoderEvent,GetMagickModule(),
             "  Reset tIME in tainted image");

          SetImageProperty(image,"date:modify", "2017-09-10T20:35:00+03:30",exception);
          timestamp=GetImageProperty(image,"date:modify",exception);
        }
2017-09-12T23:35:56+02:00 0:01.037 0.703u 7.0.6 Coder convert[8956]: png.c/WriteOnePNGImage/11107/Coder
    Reset tIME in tainted image
2017-09-12T23:35:56+02:00 0:01.039 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8152/Coder
    Writing tIME chunk: timestamp property is      2017-09-10T20:35:00+03:30

2017-09-12T23:35:56+02:00 0:01.040 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8160/Coder
     Date format specified for png:tIME=2017-09-10T20:35:00+03:30
2017-09-12T23:35:56+02:00 0:01.041 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8162/Coder
        ret=8,y=2017, m=9, d=10, h=20, m=35, s=0, ah=3, as=30
2017-09-12T23:35:56+02:00 0:01.043 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8235/Coder
        png_set_tIME: y=2017, m=9, d=10, h=24, m=5, s=0, ah=3, am=30
Member

dlemstra commented Sep 12, 2017

I cannot reproduce it on the command line anymore. But I can force the exception by changing the code to this:

      else
        {
          (void) LogMagickEvent(CoderEvent,GetMagickModule(),
             "  Reset tIME in tainted image");

          SetImageProperty(image,"date:modify", "2017-09-10T20:35:00+03:30",exception);
          timestamp=GetImageProperty(image,"date:modify",exception);
        }
2017-09-12T23:35:56+02:00 0:01.037 0.703u 7.0.6 Coder convert[8956]: png.c/WriteOnePNGImage/11107/Coder
    Reset tIME in tainted image
2017-09-12T23:35:56+02:00 0:01.039 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8152/Coder
    Writing tIME chunk: timestamp property is      2017-09-10T20:35:00+03:30

2017-09-12T23:35:56+02:00 0:01.040 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8160/Coder
     Date format specified for png:tIME=2017-09-10T20:35:00+03:30
2017-09-12T23:35:56+02:00 0:01.041 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8162/Coder
        ret=8,y=2017, m=9, d=10, h=20, m=35, s=0, ah=3, as=30
2017-09-12T23:35:56+02:00 0:01.043 0.703u 7.0.6 Coder convert[8956]: png.c/write_tIME_chunk/8235/Coder
        png_set_tIME: y=2017, m=9, d=10, h=24, m=5, s=0, ah=3, am=30
@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 12, 2017

Contributor

It is writing hour=24 while the maximum valid hour si 23. Fixed in the git repos. Also it is making the half-hour offset correction in the wrong direction; fixed that too.

Contributor

glennrp commented Sep 12, 2017

It is writing hour=24 while the maximum valid hour si 23. Fixed in the git repos. Also it is making the half-hour offset correction in the wrong direction; fixed that too.

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 13, 2017

Contributor

You're right, minute=60 is also invalid. Fixed.

Contributor

glennrp commented Sep 13, 2017

You're right, minute=60 is also invalid. Fixed.

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Sep 13, 2017

wiz
ImageMagick: update to 7.0.7.2.
2017-09-11  7.0.7-2 Glenn Randers-Pehrson <glennrp@image...>
  * Use signed integer arithmetic to caluculate timezone corrections (reference
    ImageMagick/ImageMagick#685).

2017-09-09  7.0.7-1 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-1, GIT revision 21065:ab2194121:20170909.

2017-09-09  7.0.7-1 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-09-05 7.0.7-1 Dirk Lemstra <dirk@lem.....org>
  * Added -define tiff:write-layers=true to add support for writing layered
    tiff files.

2017-09-03  7.0.7-0 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.7-0, GIT revision 20996:2f8ac2203:20170903.

2017-08-28  7.0.7-0 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Don't overwrite symbolic links when the shred policy is enabled.

2017-08-27  7.0.6-10 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-10, GIT revision 20920:9940c367a:20170827.

2017-08-27  7.0.6-10 Cristy  <quetzlzacatenango@image...>
  * Support -metric ssim, structual similarity index.

2017-08-26  7.0.6-10 Dirk Lemstra <dirk@lem.....org>
  * Fixed thread safety issue inside the pango and librsvg decoder
    (reference: dlemstra/Magick.NET#91).

2017-08-20  7.0.6-9 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-9, GIT revision 20860:3f307d8ad:20170820.

2017-08-18  7.0.6-9 Glenn Randers-Pehrson <glennrp@image...>
  * Fixed bug with writing tIME chunk when timezone has a negative offset
    (reference: ImageMagick/ImageMagick#685)

2017-08-18  7.0.6-8 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-8, GIT revision 20838:e2eb79427:20170818.

2017-08-14  7.0.6-7 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Support CubicSpline resize filter.  Define the lobes with the
    -define filter:lobes={2,3,4} (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=2&t=32506).
  * Prevent assertion failure when creating PDF thumbnail (reference
    ImageMagick/ImageMagick#674).

2017-08-12  7.0.6-7 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-7, GIT revision 20799:0db4d8a16:20170812.

2017-08-12  7.0.6-7 Cristy  <quetzlzacatenango@image...>
  * Improve EPS aliasing (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32497).

2017-08-11  7.0.6-7 Dirk Lemstra <dirk@lem.....org>
  * Added a new option called 'dds:fast-mipmaps' (reference
    ImageMagick/ImageMagick#558)
  * The mipmaps of a dds image can now be created from a list of images with
    -define dds:mipmaps=fromlist (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=2&t=30236).

2017-08-10  7.0.6-6 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-6, GIT revision 20775:061d0fa25:20170810.

2017-08-10  7.0.6-6 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-08-10  7.0.6-6 Glenn Randers-Pehrson <glennrp@image...>
  * tests/validate.c: Show the reason for failures in the test logs,
    if available.

2017-08-03  7.0.6-6 Glenn Randers-Pehrson <glennrp@image...>
  * Put UTC time in the PNG tIME chunk instead of local time (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32447).

2017-08-02  7.0.6-5 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-5, GIT revision 20715:26b28d50a:20170802.

2017-08-01  7.0.6-5 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-07-29  7.0.6-5 Glenn Randers-Pehrson <glennrp@image...>
  * Properly set image->colorspace in the PNG decoder (previously
    it was setting image->gamma, but only setting image->colorspace
    for grayscale and gray-alpha images.  Reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32418).
  * Fix improper use of NULL in the JNG decoder (CVE-2017-11750, Reference
    ImageMagick/ImageMagick#632).
  * Added "-define png:ignore-crc" option to PNG decoder. When you know
    your image has no CRC or ADLER32 errors, this can speed up decoding.
    It is also helpful in debugging bug reports from "fuzzers".

2017-07-29  7.0.6-5 Cristy  <quetzlzacatenango@image...>
  * Off by one error for gradient coder (reference
    https://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=32416),
    ImageMagick/ImageMagick#612).

2017-07-28  7.0.6-4 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-4, GIT revision 20657:4e81160d6:20170728.

2017-07-24  7.0.6-4 Cristy  <quetzlzacatenango@image...>
  * YUV coder no longer renders streaks (reference
    ImageMagick/ImageMagick#612).
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues) including
    ImageMagick/ImageMagick#618 (CVE-2017-12676).
  * coders/png.c: Initialized quantum_scanline and quantum_info
    to prevent a bad free (reference
    ImageMagick/ImageMagick#621).

2017-07-25  7.0.6-4 Glenn Randers-Pehrson <glennrp@image...>
  * Removed write_chunk_from_profile() from coders/png.c because it has
    not worked at least since version 6.7.6.
  * Removed many redundant checks before RelinquishMagickMemory(), which
    is safe to call with a NULL argument.
  * Added experimental PNG orNT chunk, to store image->orientation.
  * Removed vpAg chunk write support (we are now writing caNv instead).

2017-07-24  7.0.6-3 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-3, GIT revision 20598:cc9c43b44:20170724.

2017-07-23  7.0.6-3 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).

2017-07-23  7.0.6-3 Glenn Randers-Pehrson <glennrp@image...>
  * Fix memory leaks when reading a malformed JNG image:
    ImageMagick/ImageMagick#600 (CVE-2017-13141),
    ImageMagick/ImageMagick#602 (CVE-2017-12565).

2017-07-21  7.0.6-2 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-2, GIT revision 20549:62fcf3d96:20170721.

2017-07-19  7.0.6-2 Cristy  <quetzlzacatenango@image...>
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * The -monochrome option no longer returns a blank canvas (reference
    ImageMagick/ImageMagick#594).
  * coders/png.c: fixed memory leak of quantum_info (CVE-2017-11539, reference
    ImageMagick/ImageMagick#582
  * coders/png.c: fixed NULL dereference when trying to write an empty MNG
    (CVE-2017-11522, reference
    ImageMagick/ImageMagick#586).

2017-07-15  7.0.6-2 Glenn Randers-Pehrson <glennrp@image...>
  * Added caNv, eXIf, and pHYs to the list of PNG chunks to be removed
    by the "-strip" option.

2017-07-15  7.0.6-1 Cristy  <quetzlzacatenango@image...>
  * Release ImageMagick version 7.0.6-1, GIT revision 20447:c2a315e10:20170715.

2017-07-13  7.0.6-1 Glenn Randers-Pehrson <glennrp@image...>
  * Implemented PNG eXIf chunk support.

2017-07-08  7.0.6-1 Cristy  <quetzlzacatenango@image...>
  * Support new -auto-threshold option.  OTSU and Triangle methods are
    currently supported.  Look for the Kapur method in the next release.
  * Fixed numerous memory leaks (reference
    https://github.com/ImageMagick/ImageMagick/issues).
  * Don't use variable float_t / double_t, bump SO (reference
    ImageMagick/ImageMagick#510).
  * Support DNG images with libraw delegate library.

2017-07-02  7.0.6-1 Glenn Randers-Pehrson <glennrp@image...>
  * Reject PNG file that is too small (under 60 bytes) to contain
    a valid image.
  * Reject JPEG file that is too small (under 107 bytes) to contain
    a valid image.
  * Reject JNG file that is too small (under 147 bytes) to contain
    a valid image.

2017-06-22  7.0.6-1 Glenn Randers-Pehrson <glennrp@image...>
  * Stop a memory leak in read_user_chunk_callback() (reference
    ImageMagick/ImageMagick#517,
    CVE 2017-11310).
@jeroen

This comment has been minimized.

Show comment
Hide comment
@jeroen

jeroen Sep 20, 2017

Contributor

Has a new IM6 release with this fix been released yet?

Contributor

jeroen commented Sep 20, 2017

Has a new IM6 release with this fix been released yet?

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 20, 2017

Contributor
Contributor

glennrp commented Sep 20, 2017

@rickrich

This comment has been minimized.

Show comment
Hide comment
@rickrich

rickrich Sep 23, 2017

$ $HOME/proj/geo/mngca-newmap -agstatic -n
convert: Ignoring invalid time value `/tmp/geo22385-imap.png'
warning/png.c/MagickPNGWarningHandler/1665.

# dnf list ImageMagick
ImageMagick.x86_64 6.9.9.13-1.fc25 @updates

rickrich commented Sep 23, 2017

$ $HOME/proj/geo/mngca-newmap -agstatic -n
convert: Ignoring invalid time value `/tmp/geo22385-imap.png'
warning/png.c/MagickPNGWarningHandler/1665.

# dnf list ImageMagick
ImageMagick.x86_64 6.9.9.13-1.fc25 @updates

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Sep 23, 2017

Contributor
Contributor

glennrp commented Sep 23, 2017

@laurentKTR

This comment has been minimized.

Show comment
Hide comment
@laurentKTR

laurentKTR Oct 13, 2017

Hi
i have the same concern with date : In france , the format is day/month/year
and today we are 13/10/2017 => 13 is an invalid month

Magick.NET-Q16-AnyCPU.dll
Additional information: Invalid date format specified for png
:tIME `xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
@ error/png.c/write_tIME_chunk/8178

laurentKTR commented Oct 13, 2017

Hi
i have the same concern with date : In france , the format is day/month/year
and today we are 13/10/2017 => 13 is an invalid month

Magick.NET-Q16-AnyCPU.dll
Additional information: Invalid date format specified for png
:tIME `xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
@ error/png.c/write_tIME_chunk/8178

@glennrp

This comment has been minimized.

Show comment
Hide comment
@glennrp

glennrp Oct 13, 2017

Contributor

I don't think that's something that can be detected or fixed in the PNG encoder. It needs to be fixed wherever SetImageProperty() records the timestamp.

Contributor

glennrp commented Oct 13, 2017

I don't think that's something that can be detected or fixed in the PNG encoder. It needs to be fixed wherever SetImageProperty() records the timestamp.

@laurentKTR

This comment has been minimized.

Show comment
Hide comment
@laurentKTR

laurentKTR Oct 14, 2017

the strange thing is that i'm getting this error only when i apply BrightnessContrast with percentage on black and white points.
i have tried other methods like InverseLevel/ContrastStretch and no error occurs when i save the image
using a header reader i got this
exif:DateTime: 2007:11:26 14:05:36 => strange format for a date
PNG-tEXt

laurentKTR commented Oct 14, 2017

the strange thing is that i'm getting this error only when i apply BrightnessContrast with percentage on black and white points.
i have tried other methods like InverseLevel/ContrastStretch and no error occurs when i save the image
using a header reader i got this
exif:DateTime: 2007:11:26 14:05:36 => strange format for a date
PNG-tEXt

@laurentKTR

This comment has been minimized.

Show comment
Hide comment
@laurentKTR

laurentKTR Oct 16, 2017

i'm adding if it can help you that if i'm calling another method before BrightnessContrast,
then this error is not thrown
ex :
magImg.AutoLevel();
magImg.BrightnessContrast(new Percentage(-10), new Percentage(-30));
=> OK
only
magImg.BrightnessContrast(new Percentage(-10), new Percentage(-30));
=> KO

laurentKTR commented Oct 16, 2017

i'm adding if it can help you that if i'm calling another method before BrightnessContrast,
then this error is not thrown
ex :
magImg.AutoLevel();
magImg.BrightnessContrast(new Percentage(-10), new Percentage(-30));
=> OK
only
magImg.BrightnessContrast(new Percentage(-10), new Percentage(-30));
=> KO

@dlemstra

This comment has been minimized.

Show comment
Hide comment
@dlemstra

dlemstra Nov 4, 2017

Member

@laurentKTR It looks like you are reading a file that has an incorrect PNG timestamp. So it looks like you are experiencing a different issue. I will close this issue now. If you need more help feel free to start a new issue here or in the Magick.NET project.

Member

dlemstra commented Nov 4, 2017

@laurentKTR It looks like you are reading a file that has an incorrect PNG timestamp. So it looks like you are experiencing a different issue. I will close this issue now. If you need more help feel free to start a new issue here or in the Magick.NET project.

@dlemstra dlemstra closed this Nov 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment