After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 446809 - Gimp does not read/write the exif data if version tag is not there
Gimp does not read/write the exif data if version tag is not there
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Plugins
git master
Other All
: Normal normal
: ---
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2007-06-12 17:30 UTC by Paul Wellner Bou
Modified: 2007-06-19 08:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Sample JPG file (34.50 KB, image/jpeg)
2007-06-12 17:30 UTC, Paul Wellner Bou
Details

Description Paul Wellner Bou 2007-06-12 17:30:01 UTC
I use Bibble for RAW processing and Bibble does not seem to write the version tag correctly to the exif data of a JPG. But all other exif data is there. So gimp should be able to recognize the exif data or at least write it to the file while saving it.

I attach a sample file created with Bibble, scaled down with Gimp 2.2. Opening and saving this file with svn gimp, I get this output:

jpeg-load: found EXIF block (12012 bytes)
jpeg-save: saving EXIF block (20 bytes)
Comment 1 Paul Wellner Bou 2007-06-12 17:30:36 UTC
Created attachment 89822 [details]
Sample JPG file
Comment 2 Raphaël Quinet 2007-06-12 18:20:31 UTC
Thanks for attaching the test image.  As discussed in #gimp, it looks like Bibble discarded some useful EXIF tags from the file, including the EXIF version tag.  GIMP relied on the presence of this tag (among others) to decide if the file contained valid EXIF data or not, so in this case it was detecting the EXIF block but ignoring it.

I have now changed the tests so that if (and only if) GIMP detects something that looks like a valid EXIF block, it will always try to process it.

2007-06-12  Raphaël Quinet  <raphael@gimp.org>

        * plug-ins/jpeg/jpeg-load.c (load_image): Instead of passing any
        file to libexif blindly, check first that some JPEG APP1 marker
        contains the EXIF header.

        * plug-ins/jpeg/jpeg-exif.c (jpeg_apply_exif_data_to_image):
        removed the tests that were checking for the presense of some EXIF
        tags such as the EXIF version before deciding if the data should
        be processed or not (bug #446809).
Comment 3 Paul Wellner Bou 2007-06-19 08:50:54 UTC
Great and thank you very much!

I mailed that to the Bibble guys and they wrote that they'll fix this issue in the next release. I hope they'll do.