Am 23.05.2016 um 08:44 schrieb C R:

Fails on export with a Plug-in crash:

"Procedure 'file-png-save-defaults' returned no return values

Strangely, if you save as a jpeg first, it exports to png just fine.

Steps to reproduce: 1. Open File
2. Export to png

Results in the above crash.

Ver: GIMP 2.9.3 Ubuntu 16.04 with gnome-shell 3.18.4

-C

Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Am 23.05.2016 um 15:00 schrieb C R:

Happens every time with this file: http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp
2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

It works without trouble.

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin > wrote:

Am 23.05.2016 um 08:44 schrieb C R: > Fails on export with a Plug-in crash: >
> "Procedure 'file-png-save-defaults' returned no return values >
>
> Strangely, if you save as a jpeg first, it exports to png just fine. >
> Steps to reproduce:
> 1. Open File
> 2. Export to png
>
> Results in the above crash.
>
> Ver: GIMP 2.9.3
> Ubuntu 16.04 with gnome-shell 3.18.4 >
> -C
Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

> _______________________________________________ > gimp-developer-list mailing list > List address: gimp-developer-list@gnome.org
> List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list > List archives: https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Works properly under gimp 2.9.3 commit 2213f29, though, oddly, when I exported it to /tmp/ it was 13 bytes shorter than the original; if I export it replacing the original, the size is retained. I don't know if that's expectable gimp/png behaviour or not.

/cm

On 05/23/16 09:00, C R wrote:

Happens every time with this file: http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp
2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin < thorsten.stettin@gmail.com> wrote:

Am 23.05.2016 um 08:44 schrieb C R:

Fails on export with a Plug-in crash:

"Procedure 'file-png-save-defaults' returned no return values

Strangely, if you save as a jpeg first, it exports to png just fine.

Steps to reproduce: 1. Open File
2. Export to png

Results in the above crash.

Ver: GIMP 2.9.3 Ubuntu 16.04 with gnome-shell 3.18.4

-C

Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership:

https://mail.gnome.org/mailman/listinfo/gimp-developer-list

List archives: https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Am 23.05.2016 um 15:00 schrieb C R:

Happens every time with this file: http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp
2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

Hi

At home in Xenial I exported your picture with save comment. The above error occurred.
Hmm.

I took a look at the comment field exactly the exif:ImageDescription: It contains an invalid UTF-8 string.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4.png | grep -i desc **
** exif:ImageDescription: ..:\w.*

After replacing the picture comment content with Gimp's standard one - valid UTF-8, I think - I could export your picture without any problem.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4a.png | grep -i desc
exif:ImageDescription: Created with GIMP*

I guess it's a bug regarding the comment string and/or character set handling. Maybe a wrong character set was using in the exif:ImageDescription field, I assume.

Greetings

Thorsten

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin > wrote:

Am 23.05.2016 um 08:44 schrieb C R: > Fails on export with a Plug-in crash: >
> "Procedure 'file-png-save-defaults' returned no return values >
>
> Strangely, if you save as a jpeg first, it exports to png just fine. >
> Steps to reproduce:
> 1. Open File
> 2. Export to png
>
> Results in the above crash.
>
> Ver: GIMP 2.9.3
> Ubuntu 16.04 with gnome-shell 3.18.4 >
> -C
Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

> _______________________________________________ > gimp-developer-list mailing list > List address: gimp-developer-list@gnome.org
> List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list > List archives: https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege.
Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

Am 23.05.2016 um 08:44 schrieb C R:

Fails on export with a Plug-in crash:

"Procedure 'file-png-save-defaults' returned no return values

Strangely, if you save as a jpeg first, it exports to png just fine.

Steps to reproduce: 1. Open File
2. Export to png

Results in the above crash.

Ver: GIMP 2.9.3 Ubuntu 16.04 with gnome-shell 3.18.4

Are you able to present the original pricture file? AFAIK you were processing it with GIMP 2.9.3.

Thanks

-C
_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Lao-Tse sagt: Nichtstun ist besser, als mit viel Mhe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schtze, die ich hte und hege.
Der eine ist die Liebe, der zweite ist die Gengsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Gengsame ist grozgig, nur der Demtige ist fhig zu herrschen.

Am 23.05.2016 um 08:44 schrieb C R:

Fails on export with a Plug-in crash:

"Procedure 'file-png-save-defaults' returned no return values

Strangely, if you save as a jpeg first, it exports to png just fine.

Steps to reproduce: 1. Open File
2. Export to png

Results in the above crash.

Ver: GIMP 2.9.3 Ubuntu 16.04 with gnome-shell 3.18.4

-C

IMHO there's some weakness handling meta data. :-(

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Lao-Tse sagt: Nichtstun ist besser, als mit viel Mhe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schtze, die ich hte und hege.
Der eine ist die Liebe, der zweite ist die Gengsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Gengsame ist grozgig, nur der Demtige ist fhig zu herrschen.

Hi,

On Mon, May 23, 2016 at 7:59 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 15:00 schrieb C R:

Happens every time with this file:

http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp
2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

Hi

At home in Xenial I exported your picture with save comment. The above error occurred.
Hmm.

I took a look at the comment field exactly the exif:ImageDescription: It contains an invalid UTF-8 string.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4.png | grep -i desc **
** exif:ImageDescription: ..:\w.*

According to the Exif spec
(http://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf), even UTF-8 should be invalid. This field is supposed to be ASCII only apparently (yes that's sucky, definitely). To use multi-byte encoding, such as UTF-8 (or others such as JIS), the right field is UserComment.

After replacing the picture comment content with Gimp's standard one - valid UTF-8, I think - I could export your picture without any problem.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4a.png | grep -i desc
exif:ImageDescription: Created with GIMP*

I guess it's a bug regarding the comment string and/or character set handling. Maybe a wrong character set was using in the exif:ImageDescription field, I assume.

All this said, it would be good if GIMP were more robust to otherwise-valid images, even with buggy metadata. In particular, it should not fail to export, but simply discard the wrong metadata field with a warning.

Could you please open a bug report about this issue so that we look into this issue?
Thanks!

Jehan

Greetings

Thorsten

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin > wrote:

Am 23.05.2016 um 08:44 schrieb C R: > Fails on export with a Plug-in crash: >
> "Procedure 'file-png-save-defaults' returned no return values >
>
> Strangely, if you save as a jpeg first, it exports to png just fine. >
> Steps to reproduce:
> 1. Open File
> 2. Export to png
>
> Results in the above crash.
>
> Ver: GIMP 2.9.3
> Ubuntu 16.04 with gnome-shell 3.18.4 >
> -C
Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

> _______________________________________________ > gimp-developer-list mailing list > List address: gimp-developer-list@gnome.org
> List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list > List archives: https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

--
Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen. Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege. Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

ZeMarmot open animation film
http://film.zemarmot.net
Patreon: https://patreon.com/zemarmot

Am 23.05.2016 um 23:07 schrieb C R:

Yes, if "save comment" is unchecked it saves fine. I never touch the comment, so it's strange that there would be something invalid in it.

Thanks for looking into this. Maybe we should create a bug report?

You have precedence. You know, I'm just a packager. :-P

In German: Bitte nach Ihnen.

But seriously: From where you got this picture? Please explain the entire workflow from your camera.

IMHO information like this is very important in order to handle issues like that:

This comes from exiftooll. The whole thing is here.

*exiftool ac_a4_document_holder_vintage_leather_brown_4..png** **ExifTool Version Number : 10.10** **File Name : ac_a4_document_holder_vintage_leather_brown_4..png** **Directory : .** **File Size : 1631 kB** **File Modification Date/Time : 2016:05:23 23:28:11+02:00** **File Access Date/Time : 2016:05:23 23:29:26+02:00** **File Inode Change Date/Time : 2016:05:23 23:28:27+02:00** **File Permissions : rw-rw-r--** **File Type : PNG** **File Type Extension : png** **MIME Type : image/png** **Image Width : 1000** **Image Height : 1000** **Bit Depth : 8** **Color Type : RGB with Alpha** **Compression : Deflate/Inflate** **Filter : Adaptive** **Interlace : Noninterlaced** **Exif Byte Order : Little-endian (Intel, II)** **Bits Per Sample : 8 8 8** **Image Description : ..:\w�** **Make : samsung** **Camera Model Name : SM-N910F** **Orientation : Horizontal (normal)** **X Resolution : 72** **Y Resolution : 72** **Resolution Unit : inches** **Software : GIMP 2.9.3** **Y Cb Cr Positioning : Centered** **Exposure Time : 1/100** **F Number : 2.2** **Exposure Program : Program AE** **ISO : 50** **Exif Version : 0220** **Date/Time Original : 2016:04:08 09:51:31** **Create Date : 2016:04:08 09:51:31** **Components Configuration : Y, Cb, Cr, -** **Shutter Speed Value : 1/100** **Aperture Value : 2.2** **Brightness Value : 4.61** **Exposure Compensation : -1/2** **Max Aperture Value : 2.2** **Metering Mode : Center-weighted average** **Light Source : Unknown** **Flash : No Flash** **Focal Length : 4.8 mm** **User Comment : ..:\w�** **Flashpix Version : 0100** **Color Space : sRGB** **Exif Image Width : 5312** **Exif Image Height : 2988** **Sensing Method : One-chip color area** **Scene Type : Directly photographed** **Exposure Mode : Auto** **White Balance : Auto** **Focal Length In 35mm Format : 31 mm** **Scene Capture Type : Standard** **Image Unique ID : H16USHH04SA** **GPS Version ID : 2.2.0.0** **Photometric Interpretation : YCbCr** **Samples Per Pixel : 3** **Thumbnail Offset : 884** **Thumbnail Length : 14883** **Date Created : 2016:05:23** **Time Created : 07:46:21-07:46** **XMP Toolkit : XMP Core 4.4.0-Exiv2** **Format : image/png** **Background Color : 255 255 255** **Pixels Per Unit X : 2835** **Pixels Per Unit Y : 2835** **Pixel Units : meters** **Modify Date : 2016:05:23 06:46:23** **Aperture : 2.2** **Date/Time Created : 2016:05:23 07:46:21-07:46** **Image Size : 1000x1000** **Megapixels : 1.0** **Scale Factor To 35 mm Equivalent: 6.5** **Shutter Speed : 1/100** **Thumbnail Image : (Binary data 14883 bytes, use -b option to extract)**
**Circle Of Confusion : 0.005 mm** **Field Of View : 60.3 deg** **Focal Length : 4.8 mm (35 mm equivalent: 31.0 mm)** **Hyperfocal Distance : 2.25 m** **Light Value : 9.9** **
**.**
*

-C

On Mon, May 23, 2016 at 6:59 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 15:00 schrieb C R:

Happens every time with this file: http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp 2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

Hi

At home in Xenial I exported your picture with save comment. The above error occurred.
Hmm.

I took a look at the comment field exactly the exif:ImageDescription: It contains an invalid UTF-8 string.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4.png | grep -i desc ** ** exif:ImageDescription: ..:\w.*

After replacing the picture comment content with Gimp's standard one - valid UTF-8, I think - I could export your picture without any problem.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4a.png | grep -i desc exif:ImageDescription: Created with GIMP*

I guess it's a bug regarding the comment string and/or character set handling. Maybe a wrong character set was using in the exif:ImageDescription field, I assume.

Greetings

Thorsten

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin >> wrote:

Am 23.05.2016 um 08:44 schrieb C R: > Fails on export with a Plug-in crash: >
> "Procedure 'file-png-save-defaults' returned no return values
>
>
> Strangely, if you save as a jpeg first, it exports to png just fine.
>
> Steps to reproduce:
> 1. Open File
> 2. Export to png
>
> Results in the above crash. >
> Ver: GIMP 2.9.3
> Ubuntu 16.04 with gnome-shell 3.18.4 >
> -C
Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

> _______________________________________________ > gimp-developer-list mailing list > List address: gimp-developer-list@gnome.org
>
> List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list > List archives:
https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
>
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives:
https://mail.gnome.org/archives/gimp-developer-list

-- Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege. Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege.
Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

Hi,

On Mon, May 23, 2016 at 11:42 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 23:07 schrieb C R:

Yes, if "save comment" is unchecked it saves fine. I never touch the comment, so it's strange that there would be something invalid in it.

Thanks for looking into this. Maybe we should create a bug report?

You have precedence. You know, I'm just a packager. :-P

In German: Bitte nach Ihnen.

But seriously: From where you got this picture? Please explain the entire workflow from your camera.

IMHO information like this is very important in order to handle issues like that:

Well from my point of view, it does not matter where the broken metadata comes from, as long as it is not generated by GIMP (in which case, yes it matters a lot! GIMP should not be able to create files with broken metadata). Of course, if that's another Free Software which created the metadata, it would be cool to report a bug there, but that's still not an issue in GIMP.

The only issue I see in GIMP here is that broken metadata should not be an export-stopper since it's a data bug which is easily catchable and bypassed (by removing the wrong metadata).

Jehan

This comes from exiftooll. The whole thing is here.

*exiftool ac_a4_document_holder_vintage_leather_brown_4..png** **ExifTool Version Number : 10.10** **File Name :
ac_a4_document_holder_vintage_leather_brown_4..png** **Directory : .** **File Size : 1631 kB** **File Modification Date/Time : 2016:05:23 23:28:11+02:00** **File Access Date/Time : 2016:05:23 23:29:26+02:00** **File Inode Change Date/Time : 2016:05:23 23:28:27+02:00** **File Permissions : rw-rw-r--** **File Type : PNG** **File Type Extension : png** **MIME Type : image/png** **Image Width : 1000** **Image Height : 1000** **Bit Depth : 8** **Color Type : RGB with Alpha** **Compression : Deflate/Inflate** **Filter : Adaptive** **Interlace : Noninterlaced** **Exif Byte Order : Little-endian (Intel, II)** **Bits Per Sample : 8 8 8** **Image Description : ..:\w�** **Make : samsung** **Camera Model Name : SM-N910F** **Orientation : Horizontal (normal)** **X Resolution : 72** **Y Resolution : 72** **Resolution Unit : inches** **Software : GIMP 2.9.3** **Y Cb Cr Positioning : Centered** **Exposure Time : 1/100** **F Number : 2.2** **Exposure Program : Program AE** **ISO : 50** **Exif Version : 0220** **Date/Time Original : 2016:04:08 09:51:31** **Create Date : 2016:04:08 09:51:31** **Components Configuration : Y, Cb, Cr, -** **Shutter Speed Value : 1/100** **Aperture Value : 2.2** **Brightness Value : 4.61** **Exposure Compensation : -1/2** **Max Aperture Value : 2.2** **Metering Mode : Center-weighted average** **Light Source : Unknown** **Flash : No Flash** **Focal Length : 4.8 mm** **User Comment : ..:\w�** **Flashpix Version : 0100** **Color Space : sRGB** **Exif Image Width : 5312** **Exif Image Height : 2988** **Sensing Method : One-chip color area** **Scene Type : Directly photographed** **Exposure Mode : Auto** **White Balance : Auto** **Focal Length In 35mm Format : 31 mm** **Scene Capture Type : Standard** **Image Unique ID : H16USHH04SA** **GPS Version ID : 2.2.0.0** **Photometric Interpretation : YCbCr** **Samples Per Pixel : 3** **Thumbnail Offset : 884** **Thumbnail Length : 14883** **Date Created : 2016:05:23** **Time Created : 07:46:21-07:46** **XMP Toolkit : XMP Core 4.4.0-Exiv2** **Format : image/png** **Background Color : 255 255 255** **Pixels Per Unit X : 2835** **Pixels Per Unit Y : 2835** **Pixel Units : meters** **Modify Date : 2016:05:23 06:46:23** **Aperture : 2.2** **Date/Time Created : 2016:05:23 07:46:21-07:46** **Image Size : 1000x1000** **Megapixels : 1.0** **Scale Factor To 35 mm Equivalent: 6.5** **Shutter Speed : 1/100** **Thumbnail Image : (Binary data 14883 bytes, use -b option to extract)**
**Circle Of Confusion : 0.005 mm** **Field Of View : 60.3 deg** **Focal Length : 4.8 mm (35 mm equivalent: 31.0 mm)** **Hyperfocal Distance : 2.25 m** **Light Value : 9.9** **
**.**
*

-C

On Mon, May 23, 2016 at 6:59 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 15:00 schrieb C R:

Happens every time with this file:

http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp 2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

Hi

At home in Xenial I exported your picture with save comment. The above error occurred.
Hmm.

I took a look at the comment field exactly the exif:ImageDescription: It contains an invalid UTF-8 string.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4.png | grep -i desc ** ** exif:ImageDescription: ..:\w.*

After replacing the picture comment content with Gimp's standard one - valid UTF-8, I think - I could export your picture without any problem.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4a.png | grep -i desc exif:ImageDescription: Created with GIMP*

I guess it's a bug regarding the comment string and/or character set handling. Maybe a wrong character set was using in the exif:ImageDescription field, I assume.

Greetings

Thorsten

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin >> wrote:

Am 23.05.2016 um 08:44 schrieb C R: > Fails on export with a Plug-in crash: >
> "Procedure 'file-png-save-defaults' returned no return values
>
>
> Strangely, if you save as a jpeg first, it exports to png just fine.
>
> Steps to reproduce:
> 1. Open File
> 2. Export to png
>
> Results in the above crash. >
> Ver: GIMP 2.9.3
> Ubuntu 16.04 with gnome-shell 3.18.4 >
> -C
Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

> _______________________________________________ > gimp-developer-list mailing list > List address: gimp-developer-list@gnome.org
>
> List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list > List archives:
https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
>
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives:
https://mail.gnome.org/archives/gimp-developer-list

-- Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege. Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

--
Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen. Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege. Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

ZeMarmot open animation film
http://film.zemarmot.net
Patreon: https://patreon.com/zemarmot

Am Montag, 23. Mai 2016, 23:08:04 schrieb Jehan Pagès:

Hi,

Hi.

[...]

** exif:ImageDescription: ..:\w.*

According to the Exif spec
(http://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf), even UTF-8 should be invalid. This field is supposed to be ASCII only apparently (yes that's sucky, definitely). To use multi-byte encoding, such as UTF-8 (or others such as JIS), the right field is UserComment.

To make matters worse, the MWG guidelines [0] state that "Exif tags documented in the Exif specification as type ASCII SHOULD be written as UTF-8".

[...]

All this said, it would be good if GIMP were more robust to otherwise-valid images, even with buggy metadata. In particular, it should not fail to export, but simply discard the wrong metadata field with a warning.

Or just copy the content verbatim, there might be a reason for it being the way it is.

Could you please open a bug report about this issue so that we look into this issue?
Thanks!

Jehan

Tobias

[...]

[0] http://metadataworkinggroup.org/pdf/mwg_guidance.pdf

Am 23.05.2016 um 23:59 schrieb Jehan Pagès:

Hi,

On Mon, May 23, 2016 at 11:42 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 23:07 schrieb C R:

Yes, if "save comment" is unchecked it saves fine. I never touch the comment, so it's strange that there would be something invalid in it.

Thanks for looking into this. Maybe we should create a bug report?

You have precedence. You know, I'm just a packager. :-P

In German: Bitte nach Ihnen.

But seriously: From where you got this picture? Please explain the entire workflow from your camera.

IMHO information like this is very important in order to handle issues like that:

Well from my point of view, it does not matter where the broken metadata comes from, as long as it is not generated by GIMP (in which case, yes it matters a lot! GIMP should not be able to create files with broken metadata). Of course, if that's another Free Software which created the metadata, it would be cool to report a bug there, but that's still not an issue in GIMP.

The only issue I see in GIMP here is that broken metadata should not be an export-stopper since it's a data bug which is easily catchable and bypassed (by removing the wrong metadata).

Jehan

Jehan,
you're right but IMHO Gimp should be bullet proof regarding to problems like this.
But I htink its not wrong to know why is it so. ;-)

This comes from exiftooll. The whole thing is here.

*exiftool ac_a4_document_holder_vintage_leather_brown_4..png** **ExifTool Version Number : 10.10** **File Name :
ac_a4_document_holder_vintage_leather_brown_4..png** **Directory : .** **File Size : 1631 kB** **File Modification Date/Time : 2016:05:23 23:28:11+02:00** **File Access Date/Time : 2016:05:23 23:29:26+02:00** **File Inode Change Date/Time : 2016:05:23 23:28:27+02:00** **File Permissions : rw-rw-r--** **File Type : PNG** **File Type Extension : png** **MIME Type : image/png** **Image Width : 1000** **Image Height : 1000** **Bit Depth : 8** **Color Type : RGB with Alpha** **Compression : Deflate/Inflate** **Filter : Adaptive** **Interlace : Noninterlaced** **Exif Byte Order : Little-endian (Intel, II)** **Bits Per Sample : 8 8 8** **Image Description : ..:\w�** **Make : samsung** **Camera Model Name : SM-N910F** **Orientation : Horizontal (normal)** **X Resolution : 72** **Y Resolution : 72** **Resolution Unit : inches** **Software : GIMP 2.9.3** **Y Cb Cr Positioning : Centered** **Exposure Time : 1/100** **F Number : 2.2** **Exposure Program : Program AE** **ISO : 50** **Exif Version : 0220** **Date/Time Original : 2016:04:08 09:51:31** **Create Date : 2016:04:08 09:51:31** **Components Configuration : Y, Cb, Cr, -** **Shutter Speed Value : 1/100** **Aperture Value : 2.2** **Brightness Value : 4.61** **Exposure Compensation : -1/2** **Max Aperture Value : 2.2** **Metering Mode : Center-weighted average** **Light Source : Unknown** **Flash : No Flash** **Focal Length : 4.8 mm** **User Comment : ..:\w�** **Flashpix Version : 0100** **Color Space : sRGB** **Exif Image Width : 5312** **Exif Image Height : 2988** **Sensing Method : One-chip color area** **Scene Type : Directly photographed** **Exposure Mode : Auto** **White Balance : Auto** **Focal Length In 35mm Format : 31 mm** **Scene Capture Type : Standard** **Image Unique ID : H16USHH04SA** **GPS Version ID : 2.2.0.0** **Photometric Interpretation : YCbCr** **Samples Per Pixel : 3** **Thumbnail Offset : 884** **Thumbnail Length : 14883** **Date Created : 2016:05:23** **Time Created : 07:46:21-07:46** **XMP Toolkit : XMP Core 4.4.0-Exiv2** **Format : image/png** **Background Color : 255 255 255** **Pixels Per Unit X : 2835** **Pixels Per Unit Y : 2835** **Pixel Units : meters** **Modify Date : 2016:05:23 06:46:23** **Aperture : 2.2** **Date/Time Created : 2016:05:23 07:46:21-07:46** **Image Size : 1000x1000** **Megapixels : 1.0** **Scale Factor To 35 mm Equivalent: 6.5** **Shutter Speed : 1/100** **Thumbnail Image : (Binary data 14883 bytes, use -b option to extract)**
**Circle Of Confusion : 0.005 mm** **Field Of View : 60.3 deg** **Focal Length : 4.8 mm (35 mm equivalent: 31.0 mm)** **Hyperfocal Distance : 2.25 m** **Light Value : 9.9** **
**.**
*

-C

On Mon, May 23, 2016 at 6:59 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 15:00 schrieb C R:

Happens every time with this file:

http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png

steps to reproduce: 1.open the downloaded file in gimp 2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png" 3.these boxes in png export are checked: "save background colour", and all checkboxes after and including Save resolution, compression level 9, and everything under advanced checked.

Hi

At home in Xenial I exported your picture with save comment. The above error occurred.
Hmm.

I took a look at the comment field exactly the exif:ImageDescription: It contains an invalid UTF-8 string.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4.png | grep -i desc ** ** exif:ImageDescription: ..:\w.*

After replacing the picture comment content with Gimp's standard one - valid UTF-8, I think - I could export your picture without any problem.

*identify -verbose ac_a4_document_holder_vintage_leather_brown_4a.png | grep -i desc exif:ImageDescription: Created with GIMP*

I guess it's a bug regarding the comment string and/or character set handling. Maybe a wrong character set was using in the exif:ImageDescription field, I assume.

Greetings

Thorsten

On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin >> wrote:

Am 23.05.2016 um 08:44 schrieb C R: > Fails on export with a Plug-in crash: >
> "Procedure 'file-png-save-defaults' returned no return values
>
>
> Strangely, if you save as a jpeg first, it exports to png just fine.
>
> Steps to reproduce:
> 1. Open File
> 2. Export to png
>
> Results in the above crash. >
> Ver: GIMP 2.9.3
> Ubuntu 16.04 with gnome-shell 3.18.4 >
> -C
Under Trusty I can't reproduce that behavior. :-(

gimp Version 2.9.3.81-0t0~ppa libgegl-0.3-0 1:0.3.6.20-0t0~ppa

Cheers

> _______________________________________________ > gimp-developer-list mailing list > List address: gimp-developer-list@gnome.org
>
> List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list > List archives:
https://mail.gnome.org/archives/gimp-developer-list

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
>
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives:
https://mail.gnome.org/archives/gimp-developer-list

-- Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege. Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

_______________________________________________ gimp-developer-list mailing list List address: gimp-developer-list@gnome.org
List membership:
https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

--
Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen. Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege. Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

_______________________________________________ gimp-developer-list mailing list
List address: gimp-developer-list@gnome.org List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list List archives: https://mail.gnome.org/archives/gimp-developer-list

Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege.
Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die Demut.
Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der Demütige ist fähig zu herrschen.

Am 23.05.2016 um 23:59 schrieb Jehan Pagès:

Hi,

On Mon, May 23, 2016 at 11:42 PM, Thorsten Stettin wrote:

Am 23.05.2016 um 23:07 schrieb C R:

Yes, if "save comment" is unchecked it saves fine. I never touch the comment, so it's strange that there would be something invalid in it.

Thanks for looking into this. Maybe we should create a bug report?

You have precedence. You know, I'm just a packager. :-P

In German: Bitte nach Ihnen.

But seriously: From where you got this picture? Please explain the entire workflow from your camera.

IMHO information like this is very important in order to handle issues like that:

Well from my point of view, it does not matter where the broken metadata comes from, as long as it is not generated by GIMP (in which case, yes it matters a lot! GIMP should not be able to create files with broken metadata). Of course, if that's another Free Software which created the metadata, it would be cool to report a bug there, but that's still not an issue in GIMP.

The only issue I see in GIMP here is that broken metadata should not be an export-stopper since it's a data bug which is easily catchable and bypassed (by removing the wrong metadata).

Jehan

Au contraire mon capitan!

To cut a long story short:

It's very important to know from where comes Gimp's lag regarding non corfirming data regardless they come from.

Cheers!