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 166688 - Exif parsing incompatible with libexif 0.6
Exif parsing incompatible with libexif 0.6
Status: RESOLVED FIXED
Product: beagle
Classification: Other
Component: General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Beagle Bugs
Beagle Bugs
: 167542 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-02-08 16:25 UTC by Tom von Schwerdtner
Modified: 2005-02-22 19:32 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Solution (2.59 KB, patch)
2005-02-18 21:16 UTC, Daniel Drake 		none Details | Review
Solution (simplified) (2.36 KB, patch)
2005-02-18 22:32 UTC, Daniel Drake 		none Details | Review
Updated against CVS (2.24 KB, patch)
2005-02-19 14:49 UTC, Daniel Drake 		none Details | Review

Description Tom von Schwerdtner 2005-02-08 16:25:57 UTC 
Distribution/Version: Ubuntu Hoary

Using libexif 0.6.x Beagle crashes on some specific images I have around,
including Nat's Easter Bunny photo
(http://nat.org/camera/img_0313-800x533-border.jpg).

F-Spot has an ExifData.cs that has been updated for 0.6, however using this in
Beagle with the appropriate namespace addition doesn't seem to solve the
problem.  My outsider opinion is that there is a problem with the creation of
the ExifData object in FilterJpeg.cs:48, since the first call to to
LookupString(..) seems to be the process killer, and no debug print's at the top
of LookupString ever get executed when this problem occours (which I think
implies that the ExifData object is not created properly, but is not null
either).  However, my C# is quite amature so I could be way off.

The related but not-all-that-informative list thread starts here:
http://mail.gnome.org/archives/dashboard-hackers/2005-February/msg00033.html
Comment 1 Daniel Drake 2005-02-18 21:16:28 UTC 
Created attachment 37663 [details] [review]
Solution

This should allow both versions to be supported
Comment 2 Daniel Drake 2005-02-18 22:32:56 UTC 
Created attachment 37665 [details] [review]
Solution (simplified)

Simplified the configure.in stuff as suggested on IRC - thanks!
Comment 3 Joe Shaw 2005-02-18 22:39:36 UTC 
It doesn't apply cleanly anymore.  Can you update it against latest CVS?
Comment 4 Tom von Schwerdtner 2005-02-18 22:59:32 UTC 
FYI I vimmed in the first patch and it appears to work fine.  Thanks :)
Comment 5 Daniel Drake 2005-02-19 14:49:27 UTC 
Created attachment 37674 [details] [review]
Updated against CVS

Now that anon cvs has caught up, here's a rediff.
Comment 6 Joe Shaw 2005-02-22 19:23:19 UTC 
looks good.  will commit soon, thanks!
Comment 7 Joe Shaw 2005-02-22 19:32:34 UTC 
*** Bug 167542 has been marked as a duplicate of this bug. ***