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 667038 - metadata issu: Error on line 41 char 1: End of element <exif:Flash> not expected in this context
metadata issu: Error on line 41 char 1: End of element <exif:Flash> not expec...
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: libgimp
git master
Other All
: Normal minor
: 2.10
Assigned To: GIMP Bugs
GIMP Bugs
: 666865 692522 710937 729229 729991 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-12-30 17:34 UTC by melchiaros
Modified: 2014-11-02 17:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
hte picture - just open it ans see the error on gdb (279.24 KB, application/x-7z-compressed)
2011-12-30 17:34 UTC, melchiaros
Details
The extracted file (277.39 KB, image/jpeg)
2013-01-27 23:34 UTC, Michael Natterer
Details

Description melchiaros 2011-12-30 17:34:36 UTC
Created attachment 204353 [details]
hte picture - just open it ans see the error on gdb

I am on openSUSE12.1 64bit with GIMP 2.7.5 git 1324799282.

I´ve opened an .jpeg which causes in gdb:

jpeg-load: found XMP packet (385 bytes)
Detaching after fork from child process 11125.
While parsing XMP metadata:
Error on line 41 char 1: End of element <exif:Flash> not expected in this context


The picture is small enought to be attached here(7z).

Additional:

I do not know if this is related, but GIMP recognize it as a picture with over 8MB size, while dolphin(KDE4.7.4) is on the line that is just under 1MB.
Comment 1 Michael Schumacher 2012-01-01 18:09:53 UTC
Opens without any problems in 2.6.11, so this is most likely caused by more recent changes.
Comment 2 Michael Natterer 2013-01-27 23:34:22 UTC
Created attachment 234569 [details]
The extracted file

It's actually smaller than the compressed version...
Comment 3 Róman Joost 2013-02-06 23:17:37 UTC
*** Bug 692522 has been marked as a duplicate of this bug. ***
Comment 4 Róman Joost 2013-02-06 23:19:52 UTC
*** Bug 666865 has been marked as a duplicate of this bug. ***
Comment 5 Róman Joost 2013-02-06 23:20:44 UTC
I can reproduce the error with a GIMP 2.8 and the attached image. Thanks a lot for the report.
Comment 6 Róman Joost 2013-02-19 01:48:01 UTC
I've looked into the parser. From my point of view, the problem is caused due to missing functionality in the parser. It is not an empty tag, but rather a subtree which <exif:Flash /> defines, eg:

         <exif:Flash rdf:parseType="Resource">                           
            <exif:Fired>False</exif:Fired>                               
            <exif:Return>0</exif:Return>                                 
         </exif:Flash>

Using the debugging output with the parser, once it's parsing <exif:Flash /> it assumes a toplevel description with a namespace. That's what is causing the error.
Comment 7 Michael Natterer 2013-10-27 13:14:19 UTC
Fixed in master:

commit 21bed1e2fb438fa5721bddb0573a724ae0024455
Author: Hartmut Kuhse <onkelhatti@gimp.org>
Date:   Sat Oct 19 18:38:01 2013 +0200

    Completely rewrite metadata handling using gexiv2
    
    Based on original patches from Hartmut Kuhse and modified
    by Michael Natterer. Changes include:
    
    - remove libexif dependency and add a hard dependency on gexiv2
    - typedef GExiv2Metadata to GimpMetadata to avoid having to
      include gexiv2 globally
    - add basic GimpMetadata handling functions to libgimpbase
    - add image and image file specific metadata functions to libgimp,
      including the exif orientation image rotate dialog
    - port plug-ins to use the new APIs
    - port file-tiff-save's UI to GtkBuilder
    - add new plug-in "metadata" to view the image's metadata
    - keep metadata around as GimpImage member in the core
    - update the image's metadata on image size, resolution and precision
      changes
    - obsolete the old metadata parasites
    - migrate the old parasites to new GimpMetadata object on XCF load
Comment 8 Michael Natterer 2014-02-28 22:09:29 UTC
*** Bug 710937 has been marked as a duplicate of this bug. ***
Comment 9 Jehan 2014-04-30 21:17:09 UTC
*** Bug 729229 has been marked as a duplicate of this bug. ***
Comment 10 Michael Natterer 2014-05-16 21:10:57 UTC
*** Bug 729991 has been marked as a duplicate of this bug. ***