On Fri, Jan 29, 2010 at 7:14 PM, Alex Dedul <rot...@gmail.com> wrote:
> On Fri, Jan 29, 2010 at 6:21 PM, Omari Stephens <x...@csail.mit.edu> wrote:
>> On 01/29/2010 01:31 PM, Alex Dedul wrote:
>>> By the way, just noticed it writes this on the console
>>>
>>> pl...@plisk ~/dl/DC $ geeqie thefun.ru-britni-9-15.jpg
>>> warning: exif tag Exif.Photo.ExifVersion has 0 elements, exif spec requests 
>>> 4
>>> exif tag Exif.Photo.ExifVersion data size mismatch
>>> warning: exif tag Exif.Photo.FlashpixVersion has 0 elements, exif spec
>>> requests 4
>>> exif tag Exif.Photo.FlashpixVersion data size mismatch
>>> warning: exif tag Exif.Photo.ColorSpace has 0 elements, exif spec requests 1
>>> exif tag Exif.Photo.ColorSpace data size mismatch
>>> exif tag unknown data size mismatch
>>> Segmentation fault
>>
>> What happens if you just run `exiv2 foo.jpg`?  What version of libexiv2
>> do you have?
>
> Here it builds with --disable-exiv2. I just tried with --enable-exiv2
> and exiv2 0.19 - works okay. So seems like issue is in the geeqie's
> internal support for exif metadata or the like.

Okay, here is the trivial patch for this. Works here with it, although
still shows those exif warnings on the console.

Index: src/exif.c
===================================================================
--- src/exif.c  (revision 1898)
+++ src/exif.c  (working copy)
@@ -1457,7 +1457,7 @@

 gint exif_item_get_integer(ExifItem *item, gint *value)
 {
-   if (!item) return FALSE;
+   if (!item || !item->data) return FALSE;

    switch (item->format)
        {


With best regards from the Soul, Alex.

------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
_______________________________________________
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Reply via email to