2005-03-30 Lutz Mueller <lutz@users.sourceforge.net>
[platform/upstream/libexif.git] / libexif / exif-data.c
index 1bb9e87..92e5857 100644 (file)
  */
 
 #include <config.h>
-#include "exif-mnote-data.h"
 
-#include "exif-data.h"
-#include "exif-ifd.h"
-#include "exif-mnote-data-priv.h"
-#include "exif-utils.h"
-#include "exif-loader.h"
-#include "jpeg-marker.h"
+#include <libexif/exif-mnote-data.h>
+#include <libexif/exif-data.h>
+#include <libexif/exif-ifd.h>
+#include <libexif/exif-mnote-data-priv.h>
+#include <libexif/exif-utils.h>
+#include <libexif/exif-loader.h>
+#include <libexif/exif-log.h>
+#include <libexif/i18n.h>
 
 #include <libexif/olympus/exif-mnote-data-olympus.h>
 #include <libexif/canon/exif-mnote-data-canon.h>
 #undef MAX
 #define MAX(a, b)  (((a) > (b)) ? (a) : (b))
 
-/* #define DEBUG */
+#if defined(__WATCOMC__) || defined(_MSC_VER)
+#      define strncasecmp strnicmp
+#endif
+
+#undef JPEG_MARKER_SOI
+#define JPEG_MARKER_SOI  0xd8
+#undef JPEG_MARKER_APP0
+#define JPEG_MARKER_APP0 0xe0
+#undef JPEG_MARKER_APP1
+#define JPEG_MARKER_APP1 0xe1
 
 static const unsigned char ExifHeader[] = {0x45, 0x78, 0x69, 0x66, 0x00, 0x00};
 
@@ -49,12 +59,29 @@ struct _ExifDataPrivate
 
        ExifMnoteData *md;
 
+       ExifLog *log;
+       ExifMem *mem;
+
        unsigned int ref_count;
 
        /* Temporarily used while loading data */
        unsigned int offset_mnote;
 };
 
+static void *
+exif_data_alloc (ExifData *data, unsigned int i)
+{
+       void *d;
+
+       if (!data || !i) return NULL;
+
+       d = exif_mem_alloc (data->priv->mem, i);
+       if (d) return d;
+
+       EXIF_LOG_NO_MEMORY (data->priv->log, "ExifData", i);
+       return NULL;
+}
+
 ExifMnoteData *
 exif_data_get_mnote_data (ExifData *d)
 {
@@ -64,23 +91,33 @@ exif_data_get_mnote_data (ExifData *d)
 ExifData *
 exif_data_new (void)
 {
+       ExifMem *mem = exif_mem_new_default ();
+       ExifData *d = exif_data_new_mem (mem);
+
+       exif_mem_unref (mem);
+
+       return d;
+}
+
+ExifData *
+exif_data_new_mem (ExifMem *mem)
+{
        ExifData *data;
        unsigned int i;
 
-       data = malloc (sizeof (ExifData));
-       if (!data)
-               return (NULL);
-       memset (data, 0, sizeof (ExifData));
-       data->priv = malloc (sizeof (ExifDataPrivate));
-       if (!data->priv) {
-               free (data);
-               return (NULL);
-       }
-       memset (data->priv, 0, sizeof (ExifDataPrivate));
+       if (!mem) return NULL;
+
+       data = exif_mem_alloc (mem, sizeof (ExifData));
+       if (!data) return (NULL);
+       data->priv = exif_mem_alloc (mem, sizeof (ExifDataPrivate));
+       if (!data->priv) { exif_mem_free (mem, data); return (NULL); }
        data->priv->ref_count = 1;
 
+       data->priv->mem = mem;
+       exif_mem_ref (mem);
+
        for (i = 0; i < EXIF_IFD_COUNT; i++) {
-               data->ifd[i] = exif_content_new ();
+               data->ifd[i] = exif_content_new_mem (data->priv->mem);
                if (!data->ifd[i]) {
                        exif_data_free (data);
                        return (NULL);
@@ -112,6 +149,10 @@ exif_data_load_data_entry (ExifData *data, ExifEntry *entry,
        entry->format     = exif_get_short (d + offset + 2, data->priv->order);
        entry->components = exif_get_long  (d + offset + 4, data->priv->order);
 
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Loading entry 0x%x ('%s')...", entry->tag,
+                 exif_tag_get_name (entry->tag));
+
        /*
         * Size? If bigger than 4 bytes, the actual data is not
         * in the entry but somewhere else (offset).
@@ -128,53 +169,88 @@ exif_data_load_data_entry (ExifData *data, ExifEntry *entry,
        if (size < doff + s)
                return;
 
-       entry->data = malloc (sizeof (char) * s);
-       if (!entry->data)
-               return;
-       entry->size = s;
-       memcpy (entry->data, d + doff, s);
+       entry->data = exif_data_alloc (data, s);
+       if (entry->data) {
+               entry->size = s;
+               memcpy (entry->data, d + doff, s);
+       }
 
        /* If this is the MakerNote, remember the offset */
-       if (entry->tag == EXIF_TAG_MAKER_NOTE)
+       if (entry->tag == EXIF_TAG_MAKER_NOTE) {
+               if (entry->size > 6) exif_log (data->priv->log,
+                         EXIF_LOG_CODE_DEBUG, "ExifData",
+                         "MakerNote found (%02x %02x %02x %02x "
+                         "%02x %02x %02x...).",
+                         entry->data[0], entry->data[1], entry->data[2],
+                         entry->data[3], entry->data[4], entry->data[5],
+                         entry->data[6]);
                data->priv->offset_mnote = doff;
+       }
+
+       exif_entry_fix (entry);
 }
 
 static void
-exif_data_save_data_entry (ExifData *data, ExifEntry *entry,
+exif_data_save_data_entry (ExifData *data, ExifEntry *e,
                           unsigned char **d, unsigned int *ds,
                           unsigned int offset)
 {
        unsigned int doff, s;
 
+       if (!data || !data->priv) return;
+
        /*
         * Each entry is 12 bytes long. The memory for the entry has
         * already been allocated.
         */
        exif_set_short (*d + 6 + offset + 0,
-                       data->priv->order, entry->tag);
+                       data->priv->order, (ExifShort) e->tag);
        exif_set_short (*d + 6 + offset + 2,
-                       data->priv->order, entry->format);
+                       data->priv->order, (ExifShort) e->format);
+
+#ifndef EXIF_DONT_CHANGE_MAKER_NOTE
+       /* If this is the maker note tag, update it. */
+       if ((e->tag == EXIF_TAG_MAKER_NOTE) && data->priv->md) {
+               exif_mem_free (data->priv->mem, e->data);
+               e->data = NULL;
+               e->size = 0;
+               exif_mnote_data_set_offset (data->priv->md, *ds - 6);
+               exif_mnote_data_save (data->priv->md, &e->data, &e->size);
+               e->components = e->size;
+       }
+#endif
+
        exif_set_long  (*d + 6 + offset + 4,
-                       data->priv->order, entry->components);
+                       data->priv->order, e->components);
 
        /*
         * Size? If bigger than 4 bytes, the actual data is not in
         * the entry but somewhere else.
         */
-       s = exif_format_get_size (entry->format) * entry->components;
-       if (!s)
-               return;
+       s = exif_format_get_size (e->format) * e->components;
        if (s > 4) {
-               *ds += entry->size;
-               *d = realloc (*d, sizeof (char) * *ds);
-               doff = *ds - 6 - entry->size;
-               exif_set_long (*d + 6 + offset + 8,
-                              data->priv->order, doff);
+               doff = *ds - 6;
+               *ds += s;
+
+               /*
+                * According to the TIFF specification,
+                * the offset must be an even number. If we need to introduce
+                * a padding byte, we set it to 0.
+                */
+               if (s & 1) (*ds)++;
+               *d = exif_mem_realloc (data->priv->mem, *d, *ds);
+               if (!*d) {
+                       EXIF_LOG_NO_MEMORY (data->priv->log, "ExifData", *ds);
+                       return;
+               }
+               exif_set_long (*d + 6 + offset + 8, data->priv->order, doff);
+               if (s & 1) *(*d + *ds - 1) = '\0';
+
        } else
                doff = offset + 8;
 
        /* Write the data. Fill unneeded bytes with 0. */
-       memcpy (*d + 6 + doff, entry->data, entry->size);
+       memcpy (*d + 6 + doff, e->data, s);
        if (s < 4) memset (*d + 6 + doff + s, 0, (4 - s));
 }
 
@@ -183,19 +259,36 @@ exif_data_load_data_thumbnail (ExifData *data, const unsigned char *d,
                               unsigned int ds, ExifLong offset, ExifLong size)
 {
        if (ds < offset + size) {
-#ifdef DEBUG
-               printf ("Bogus thumbnail offset and size: %i < %i + %i.\n",
-                       (int) ds, (int) offset, (int) size);
-#endif
+               exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                         "Bogus thumbnail offset and size: %i < %i + %i.",
+                         (int) ds, (int) offset, (int) size);
                return;
        }
-       if (data->data)
-               free (data->data);
+       if (data->data) exif_mem_free (data->priv->mem, data->data);
        data->size = size;
-       data->data = malloc (sizeof (char) * data->size);
+       data->data = exif_data_alloc (data, data->size);
+       if (!data->data) return;
        memcpy (data->data, d + offset, data->size);
 }
 
+#undef CHECK_REC
+#define CHECK_REC(i)                                   \
+if (data->ifd[(i)] == ifd) {                           \
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, \
+               "ExifData", "Recursive entry in IFD "   \
+               "'%s' detected. Skipping...",           \
+               exif_ifd_get_name (i));                 \
+       break;                                          \
+}                                                      \
+if (data->ifd[(i)]->count) {                           \
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, \
+               "ExifData", "Attemt to load IFD "       \
+               "'%s' multiple times detected. "        \
+               "Skipping...",                          \
+               exif_ifd_get_name (i));                 \
+       break;                                          \
+}
+
 static void
 exif_data_load_data_content (ExifData *data, ExifContent *ifd,
                             const unsigned char *d,
@@ -207,12 +300,13 @@ exif_data_load_data_content (ExifData *data, ExifContent *ifd,
        unsigned int i;
        ExifTag tag;
 
+       if (!data || !data->priv) return;
+
        /* Read the number of entries */
        if (offset >= ds - 1) return;
        n = exif_get_short (d + offset, data->priv->order);
-#ifdef DEBUG
-       printf ("Loading %i entries...\n", n);
-#endif
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Loading %i entries...", n);
        offset += 2;
 
        /* Check if we have enough data. */
@@ -221,10 +315,6 @@ exif_data_load_data_content (ExifData *data, ExifContent *ifd,
        for (i = 0; i < n; i++) {
 
                tag = exif_get_short (d + offset + 12 * i, data->priv->order);
-#ifdef DEBUG
-               printf ("Loading entry '%s' (%i of %i)...\n",
-                       exif_tag_get_name (tag), i + 1, n);
-#endif
                switch (tag) {
                case EXIF_TAG_EXIF_IFD_POINTER:
                case EXIF_TAG_GPS_INFO_IFD_POINTER:
@@ -235,21 +325,21 @@ exif_data_load_data_content (ExifData *data, ExifContent *ifd,
                                           data->priv->order);
                        switch (tag) {
                        case EXIF_TAG_EXIF_IFD_POINTER:
+                               CHECK_REC (EXIF_IFD_EXIF);
                                exif_data_load_data_content (data,
                                        data->ifd[EXIF_IFD_EXIF], d, ds, o);
                                break;
                        case EXIF_TAG_GPS_INFO_IFD_POINTER:
+                               CHECK_REC (EXIF_IFD_GPS);
                                exif_data_load_data_content (data,
                                        data->ifd[EXIF_IFD_GPS], d, ds, o);
                                break;
                        case EXIF_TAG_INTEROPERABILITY_IFD_POINTER:
+                               CHECK_REC (EXIF_IFD_INTEROPERABILITY);
                                exif_data_load_data_content (data,
                                        data->ifd[EXIF_IFD_INTEROPERABILITY], d, ds, o);
                                break;
                        case EXIF_TAG_JPEG_INTERCHANGE_FORMAT:
-#ifdef DEBUG
-                               printf ("Thumbnail at %i.\n", (int) o);
-#endif
                                thumbnail_offset = o;
                                if (thumbnail_offset && thumbnail_length)
                                        exif_data_load_data_thumbnail (data, d,
@@ -257,9 +347,6 @@ exif_data_load_data_content (ExifData *data, ExifContent *ifd,
                                                thumbnail_length);
                                break;
                        case EXIF_TAG_JPEG_INTERCHANGE_FORMAT_LENGTH:
-#ifdef DEBUG
-                               printf ("Thumbnail size: %i.\n", (int) o);
-#endif
                                thumbnail_length = o;
                                if (thumbnail_offset && thumbnail_length)
                                        exif_data_load_data_thumbnail (data, d,
@@ -271,16 +358,49 @@ exif_data_load_data_content (ExifData *data, ExifContent *ifd,
                        }
                        break;
                default:
-                       entry = exif_entry_new ();
-                       exif_content_add_entry (ifd, entry);
+
+                       /*
+                        * If we don't know the tag, chances are high
+                        * that the EXIF data does not follow the standard.
+                        */
+                       if (!exif_tag_get_name (tag)) {
+                               exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                                 "Unknown tag %x (entry %i)", tag, i);
+                               return;
+                       }
+                       entry = exif_entry_new_mem (data->priv->mem);
                        exif_data_load_data_entry (data, entry, d, ds,
                                                   offset + 12 * i);
+                       exif_content_add_entry (ifd, entry);
                        exif_entry_unref (entry);
                        break;
                }
        }
 }
 
+static int
+cmp_func (const unsigned char *p1, const unsigned char *p2, ExifByteOrder o)
+{
+       ExifShort tag1 = exif_get_short (p1, o);
+       ExifShort tag2 = exif_get_short (p2, o);
+
+       return (tag1 < tag2) ? -1 : (tag1 > tag2) ? 1 : 0;
+}
+
+static int
+cmp_func_intel (const void *elem1, const void *elem2)
+{
+       return cmp_func ((const unsigned char *) elem1,
+                       (const unsigned char *) elem2, EXIF_BYTE_ORDER_INTEL);
+}
+
+static int
+cmp_func_motorola (const void *elem1, const void *elem2)
+{
+       return cmp_func ((const unsigned char *) elem1,
+                       (const unsigned char *) elem2, EXIF_BYTE_ORDER_MOTOROLA);
+}
+
 static void
 exif_data_save_data_content (ExifData *data, ExifContent *ifd,
                             unsigned char **d, unsigned int *ds,
@@ -289,8 +409,7 @@ exif_data_save_data_content (ExifData *data, ExifContent *ifd,
        unsigned int j, n_ptr = 0, n_thumb = 0;
        ExifIfd i;
 
-       if (!data || !ifd || !d || !ds)
-               return;
+       if (!data || !data->priv || !ifd || !d || !ds) return;
 
        for (i = 0; i < EXIF_IFD_COUNT; i++)
                if (ifd == data->ifd[i])
@@ -333,22 +452,24 @@ exif_data_save_data_content (ExifData *data, ExifContent *ifd,
         * and the number of entries.
         */
        *ds += (2 + (ifd->count + n_ptr + n_thumb) * 12 + 4);
-       *d = realloc (*d, sizeof (char) * *ds);
+       *d = exif_mem_realloc (data->priv->mem, *d, *ds);
+       if (!*d) {
+               EXIF_LOG_NO_MEMORY (data->priv->log, "ExifData", *ds);
+               return;
+       }
 
        /* Save the number of entries */
        exif_set_short (*d + 6 + offset, data->priv->order,
-                       ifd->count + n_ptr + n_thumb);
+                       (ExifShort) (ifd->count + n_ptr + n_thumb));
        offset += 2;
 
-#ifdef DEBUG
-       printf ("Saving %i entries (IFD '%s', offset: %i)...\n",
-               ifd->count, exif_ifd_get_name (i), offset);
-#endif
-
        /* Save each entry */
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Saving %i entries (IFD '%s', offset: %i)...",
+                 ifd->count, exif_ifd_get_name (i), offset);
        for (j = 0; j < ifd->count; j++)
-               exif_data_save_data_entry (data, ifd->entries[j],
-                               d, ds, offset + 12 * j);
+               exif_data_save_data_entry (data, ifd->entries[j], d, ds, offset + 12 * j);
+
        offset += 12 * ifd->count;
 
        /* Now save special entries. */
@@ -432,14 +553,14 @@ exif_data_save_data_content (ExifData *data, ExifContent *ifd,
                        exif_set_long  (*d + 6 + offset + 8, data->priv->order,
                                        *ds - 6);
                        *ds += data->size;
-                       *d = realloc (*d, sizeof (char) * *ds);
+                       *d = exif_mem_realloc (data->priv->mem, *d, *ds);
+                       if (!*d) {
+                               EXIF_LOG_NO_MEMORY (data->priv->log, "ExifData",
+                                                   *ds);
+                               return;
+                       }
                        memcpy (*d + *ds - data->size, data->data, data->size);
                        offset += 12;
-#ifdef DEBUG
-                       printf ("Wrote %i bytes of thumbnail data at offset "
-                               "%i.\n", data->size, *ds - data->size);
-                       printf ("We currently have %i bytes EXIF data.\n", *ds);
-#endif
 
                        /* EXIF_TAG_JPEG_INTERCHANGE_FORMAT_LENGTH */
                        exif_set_short (*d + 6 + offset + 0, data->priv->order,
@@ -458,6 +579,11 @@ exif_data_save_data_content (ExifData *data, ExifContent *ifd,
                break;
        }
 
+       /* Sort the directory according to TIFF specification */
+       qsort (*d + 6 + offset - (ifd->count + n_ptr + n_thumb) * 12,
+                       (ifd->count + n_ptr + n_thumb), 12,
+                       data->priv->order == EXIF_BYTE_ORDER_INTEL ? cmp_func_intel : cmp_func_motorola);
+
        /* Correctly terminate the directory */
        if (i == EXIF_IFD_0 && (data->ifd[EXIF_IFD_1]->count ||
                                             data->size)) {
@@ -473,21 +599,53 @@ exif_data_save_data_content (ExifData *data, ExifContent *ifd,
                exif_set_long (*d + 6 + offset, data->priv->order, 0);
 }
 
-static void
-exif_data_remove_entry (ExifData *d, ExifTag t)
+typedef enum {
+       EXIF_DATA_TYPE_MAKER_NOTE_NONE          = 0,
+       EXIF_DATA_TYPE_MAKER_NOTE_CANON         = 1,
+       EXIF_DATA_TYPE_MAKER_NOTE_OLYMPUS       = 2,
+       EXIF_DATA_TYPE_MAKER_NOTE_PENTAX        = 3
+} ExifDataTypeMakerNote;
+
+static ExifDataTypeMakerNote
+exif_data_get_type_maker_note (ExifData *d)
 {
-       exif_content_remove_entry (d->ifd[EXIF_IFD_0],
-               exif_content_get_entry (d->ifd[EXIF_IFD_0], t));
-       exif_content_remove_entry (d->ifd[EXIF_IFD_1],
-               exif_content_get_entry (d->ifd[EXIF_IFD_1], t));
-       exif_content_remove_entry (d->ifd[EXIF_IFD_EXIF],
-               exif_content_get_entry (d->ifd[EXIF_IFD_EXIF], t));
-       exif_content_remove_entry (d->ifd[EXIF_IFD_GPS],
-               exif_content_get_entry (d->ifd[EXIF_IFD_GPS], t));
-       exif_content_remove_entry (d->ifd[EXIF_IFD_INTEROPERABILITY],
-               exif_content_get_entry (d->ifd[EXIF_IFD_INTEROPERABILITY], t));
+       ExifEntry *e, *em;
+       char value[1024];
+
+       if (!d) return EXIF_DATA_TYPE_MAKER_NOTE_NONE;
+       
+       e = exif_data_get_entry (d, EXIF_TAG_MAKER_NOTE);
+        if (!e) return EXIF_DATA_TYPE_MAKER_NOTE_NONE;
+
+       /* Olympus & Nikon */
+       if ((e->size >= 5) && (!memcmp (e->data, "OLYMP", 5) ||
+                              !memcmp (e->data, "Nikon", 5)))
+               return EXIF_DATA_TYPE_MAKER_NOTE_OLYMPUS;
+
+       em = exif_data_get_entry (d, EXIF_TAG_MAKE);
+       if (!em) return EXIF_DATA_TYPE_MAKER_NOTE_NONE;
+
+       /* Canon */
+       if (!strcmp (exif_entry_get_value (em, value, sizeof (value)), "Canon"))
+               return EXIF_DATA_TYPE_MAKER_NOTE_CANON;
+
+       /* Pentax & some variant of Nikon */
+       if ((e->size >= 2) && (e->data[0] == 0x00) && (e->data[1] == 0x1b)) {
+               if (!strncasecmp (
+                       exif_entry_get_value (em, value, sizeof(value)),
+                                             "Nikon", 5))
+                       return EXIF_DATA_TYPE_MAKER_NOTE_OLYMPUS;
+               else
+                       return EXIF_DATA_TYPE_MAKER_NOTE_PENTAX;
+       }
+
+       return EXIF_DATA_TYPE_MAKER_NOTE_NONE;
 }
 
+#define LOG_TOO_SMALL \
+exif_log (data->priv->log, EXIF_LOG_CODE_CORRUPT_DATA, "ExifData", \
+               _("Size of data too small to allow for EXIF data."));
+
 void
 exif_data_load_data (ExifData *data, const unsigned char *d_orig,
                     unsigned int ds_orig)
@@ -495,38 +653,26 @@ exif_data_load_data (ExifData *data, const unsigned char *d_orig,
        unsigned int l;
        ExifLong offset;
        ExifShort n;
-       ExifEntry *e, *em;
        const unsigned char *d = d_orig;
        unsigned int ds = ds_orig, len;
 
-       if (!data)
-               return;
-       if (!d || !ds)
-               return;
+       if (!data || !data->priv || !d || !ds) return;
 
-#ifdef DEBUG
-       printf ("Parsing %i byte(s) EXIF data...\n", ds);
-#endif
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Parsing %i byte(s) EXIF data...\n", ds);
 
        /*
         * It can be that the data starts with the EXIF header. If it does
         * not, search the EXIF marker.
         */
        if (ds < 6) {
-#ifdef DEBUG
-               printf ("Size too small.\n");
-#endif
+               LOG_TOO_SMALL;
                return;
        }
        if (!memcmp (d, ExifHeader, 6)) {
-#ifdef DEBUG
-               printf ("Found EXIF header.\n");
-#endif
+               exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                         "Found EXIF header.");
        } else {
-#ifdef DEBUG
-               printf ("Data begins with 0x%x 0x%x 0x%x 0x%x 0x%x 0x%x "
-                       "0x%x...\n", d[0], d[1], d[2], d[3], d[4], d[5], d[6]);
-#endif
                while (1) {
                        while ((d[0] == 0xff) && ds) {
                                d++;
@@ -557,23 +703,20 @@ exif_data_load_data (ExifData *data, const unsigned char *d_orig,
                                break;
 
                        /* Unknown marker or data. Give up. */
-#ifdef DEBUG
-                       printf ("EXIF marker not found.\n");
-#endif
+                       exif_log (data->priv->log, EXIF_LOG_CODE_CORRUPT_DATA,
+                                 "ExifData", _("EXIF marker not found."));
                        return;
                }
                d++;
                ds--;
                if (ds < 2) {
-#ifdef DEBUG
-                       printf ("Size too small.\n");
-#endif
+                       LOG_TOO_SMALL;
                        return;
                }
                len = (d[0] << 8) | d[1];
-#ifdef DEBUG
-               printf ("We have to deal with %i byte(s) of EXIF data.\n", len);
-#endif
+               exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                         "We have to deal with %i byte(s) of EXIF data.",
+                         len);
                d += 2;
                ds -= 2;
        }
@@ -583,31 +726,30 @@ exif_data_load_data (ExifData *data, const unsigned char *d_orig,
         * (offset 2, length 6).
         */
        if (ds < 6) {
-#ifdef DEBUG
-               printf ("Size too small.\n");
-#endif
+               LOG_TOO_SMALL;
                return;
        }
        if (memcmp (d, ExifHeader, 6)) {
-#ifdef DEBUG
-               printf ("EXIF header not found.\n");
-#endif
+               exif_log (data->priv->log, EXIF_LOG_CODE_CORRUPT_DATA,
+                               "ExifData", _("EXIF header not found."));
                return;
        }
 
-#ifdef DEBUG
-       printf ("Found EXIF header.\n");
-#endif
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Found EXIF header.");
 
        /* Byte order (offset 6, length 2) */
-       if (ds < 12)
+       if (ds < 14)
                return;
        if (!memcmp (d + 6, "II", 2))
                data->priv->order = EXIF_BYTE_ORDER_INTEL;
        else if (!memcmp (d + 6, "MM", 2))
                data->priv->order = EXIF_BYTE_ORDER_MOTOROLA;
-       else
+       else {
+               exif_log (data->priv->log, EXIF_LOG_CODE_CORRUPT_DATA,
+                               "ExifData", _("Unknown encoding."));
                return;
+       }
 
        /* Fixed value */
        if (exif_get_short (d + 8, data->priv->order) != 0x002a)
@@ -615,27 +757,30 @@ exif_data_load_data (ExifData *data, const unsigned char *d_orig,
 
        /* IFD 0 offset */
        offset = exif_get_long (d + 10, data->priv->order);
-#ifdef DEBUG
-       printf ("IFD 0 at %i.\n", (int) offset);
-#endif
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData", 
+                 "IFD 0 at %i.", (int) offset);
 
-       /* Parse the actual exif data (offset 14) */
+       /* Parse the actual exif data (usually offset 14 from start) */
        exif_data_load_data_content (data, data->ifd[EXIF_IFD_0], d + 6,
                                     ds - 6, offset);
 
        /* IFD 1 offset */
+       if (offset + 6 + 2 > ds) {
+               return;
+       }
        n = exif_get_short (d + 6 + offset, data->priv->order);
+       if (offset + 6 + 2 + 12 * n + 4 > ds) {
+               return;
+       }
        offset = exif_get_long (d + 6 + offset + 2 + 12 * n, data->priv->order);
        if (offset) {
-#ifdef DEBUG
-               printf ("IFD 1 at %i.\n", (int) offset);
-#endif
+               exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                         "IFD 1 at %i.", (int) offset);
 
                /* Sanity check. */
                if (offset > ds - 6) {
-#ifdef DEBUG
-                       printf ("Bogus offset!\n");
-#endif
+                       exif_log (data->priv->log, EXIF_LOG_CODE_CORRUPT_DATA,
+                                 "ExifData", "Bogus offset.");
                        return;
                }
 
@@ -649,60 +794,46 @@ exif_data_load_data (ExifData *data, const unsigned char *d_orig,
         * space between IFDs. Here is the only place where we have access
         * to that data.
         */
-       e = exif_data_get_entry (data, EXIF_TAG_MAKER_NOTE);
-       if (e) {
-
-           /* Olympus */
-           if ((e->size >= 5) && (!memcmp (e->data, "OLYMP", 5)))
-               data->priv->md = exif_mnote_data_olympus_new ();
-
-           /* Pentax */
-           else if ((e->size >= 2) && (e->data[0] == 0x00)
-                                   && (e->data[1] == 0x1b))
-               data->priv->md = exif_mnote_data_pentax_new ();
-
-           else {
-               em = exif_data_get_entry (data, EXIF_TAG_MAKE);
-               if (em) {
+       switch (exif_data_get_type_maker_note (data)) {
+       case EXIF_DATA_TYPE_MAKER_NOTE_OLYMPUS:
+               data->priv->md = exif_mnote_data_olympus_new (data->priv->mem);
+               break;
+       case EXIF_DATA_TYPE_MAKER_NOTE_PENTAX:
+               data->priv->md = exif_mnote_data_pentax_new (data->priv->mem);
+               break;
+       case EXIF_DATA_TYPE_MAKER_NOTE_CANON:
+               data->priv->md = exif_mnote_data_canon_new (data->priv->mem);
+               break;
+       default:
+               break;
+       }
 
-                   /* Canon */
-                   if (!strcmp (exif_entry_get_value (em), "Canon"))
-                       data->priv->md = exif_mnote_data_canon_new ();
-               }
-           }
-
-           /* 
-            * If we are able to interpret the maker note, interpret it and
-            * remove the corresponding entry as it may contain invalid
-            * pointers after this function here returns.
-            */
-           if (data->priv->md) {
+       /* 
+        * If we are able to interpret the maker note, do so.
+        */
+       if (data->priv->md) {
+               exif_mnote_data_log (data->priv->md, data->priv->log);
                exif_mnote_data_set_byte_order (data->priv->md,
                                                data->priv->order);
                exif_mnote_data_set_offset (data->priv->md,
                                            data->priv->offset_mnote);
                exif_mnote_data_load (data->priv->md, d, ds);
-               exif_data_remove_entry (data, EXIF_TAG_MAKER_NOTE);
-           }
        }
 }
 
 void
 exif_data_save_data (ExifData *data, unsigned char **d, unsigned int *ds)
 {
-       if (!data)
-               return;
-       if (!d || !ds)
+       if (!data || !d || !ds)
                return;
 
        /* Header */
-       *ds = 6;
-       *d = malloc (sizeof (char) * *ds);
+       *ds = 14;
+       *d = exif_data_alloc (data, *ds);
+       if (!*d) return;
        memcpy (*d, ExifHeader, 6);
 
        /* Order (offset 6) */
-       *ds += 2;
-       *d = realloc (*d, sizeof (char) * *ds);
        if (data->priv->order == EXIF_BYTE_ORDER_INTEL) {
                memcpy (*d + 6, "II", 2);
        } else {
@@ -710,8 +841,6 @@ exif_data_save_data (ExifData *data, unsigned char **d, unsigned int *ds)
        }
 
        /* Fixed value (2 bytes, offset 8) */
-       *ds += 2;
-       *d = realloc (*d, sizeof (char) * *ds);
        exif_set_short (*d + 8, data->priv->order, 0x002a);
 
        /*
@@ -720,43 +849,25 @@ exif_data_save_data (ExifData *data, unsigned char **d, unsigned int *ds)
         * EXIF header (2 bytes for order, another 2 for the test, and
         * 4 bytes for the IFD 0 offset make 8 bytes together).
         */
-       *ds += 4;
-       *d = realloc (*d, sizeof (char) * *ds);
        exif_set_long (*d + 10, data->priv->order, 8);
 
        /* Now save IFD 0. IFD 1 will be saved automatically. */
-#ifdef DEBUG
-       printf ("Saving IFDs...\n");
-#endif
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Saving IFDs...");
        exif_data_save_data_content (data, data->ifd[EXIF_IFD_0], d, ds,
                                     *ds - 6);
-
-#ifdef DEBUG
-       printf ("Saved %i byte(s) EXIF data.\n", *ds);
-#endif
+       exif_log (data->priv->log, EXIF_LOG_CODE_DEBUG, "ExifData",
+                 "Saved %i byte(s) EXIF data.", *ds);
 }
 
 ExifData *
 exif_data_new_from_file (const char *path)
 {
-       FILE *f;
-       int size;
        ExifData *edata;
        ExifLoader *loader;
-       unsigned char data[1024];
-
-       f = fopen (path, "rb");
-       if (!f)
-               return (NULL);
 
        loader = exif_loader_new ();
-       while (1) {
-               size = fread (data, 1, 1024, f);
-               if (size <= 0) break;
-               if (!exif_loader_write (loader, data, size)) break;
-       }
-       fclose (f);
-
+       exif_loader_write_file (loader, path);
        edata = exif_loader_get_data (loader);
        exif_loader_unref (loader);
 
@@ -775,21 +886,19 @@ exif_data_ref (ExifData *data)
 void
 exif_data_unref (ExifData *data)
 {
-       if (!data)
-               return;
+       if (!data) return;
 
        data->priv->ref_count--;
-       if (!data->priv->ref_count)
-               exif_data_free (data);
+       if (!data->priv->ref_count) exif_data_free (data);
 }
 
 void
 exif_data_free (ExifData *data)
 {
        unsigned int i;
+       ExifMem *mem = (data && data->priv) ? data->priv->mem : NULL;
 
-       if (!data)
-               return;
+       if (!data) return;
 
        for (i = 0; i < EXIF_IFD_COUNT; i++) {
                if (data->ifd[i]) {
@@ -797,19 +906,26 @@ exif_data_free (ExifData *data)
                        data->ifd[i] = NULL;
                }
        }
+
        if (data->data) {
-               free (data->data);
+               exif_mem_free (mem, data->data);
                data->data = NULL;
        }
+
        if (data->priv) {
+               if (data->priv->log) {
+                       exif_log_unref (data->priv->log);
+                       data->priv->log = NULL;
+               }
                if (data->priv->md) {
                        exif_mnote_data_unref (data->priv->md);
                        data->priv->md = NULL;
                }
-               free (data->priv);
-               data->priv = NULL;
+               exif_mem_free (mem, data->priv);
+               exif_mem_free (mem, data);
        }
-       free (data);
+
+       exif_mem_unref (mem);
 }
 
 void
@@ -870,74 +986,11 @@ static void
 entry_set_byte_order (ExifEntry *e, void *data)
 {
        ByteOrderChangeData *d = data;
-       unsigned int i;
-       ExifShort s;
-       ExifLong l;
-       ExifSLong sl;
-       ExifRational r;
-       ExifSRational sr;
 
        if (!e)
                return;
 
-       switch (e->format) {
-       case EXIF_FORMAT_SHORT:
-               for (i = 0; i < e->components; i++) {
-                       s = exif_get_short (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->old);
-                       exif_set_short (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->new, s);
-               }
-               break;
-       case EXIF_FORMAT_LONG:
-               for (i = 0; i < e->components; i++) {
-                       l = exif_get_long (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->old);
-                       exif_set_long (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->new, l);
-               }
-               break;
-       case EXIF_FORMAT_RATIONAL:
-               for (i = 0; i < e->components; i++) {
-                       r = exif_get_rational (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->old);
-                       exif_set_rational (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->new, r);
-               }
-               break;
-       case EXIF_FORMAT_SLONG:
-               for (i = 0; i < e->components; i++) {
-                       sl = exif_get_slong (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->old);
-                       exif_set_slong (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->new, sl);
-               }
-               break;
-       case EXIF_FORMAT_SRATIONAL:
-               for (i = 0; i < e->components; i++) {
-                       sr = exif_get_srational (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->old);
-                       exif_set_srational (e->data +
-                               (i * exif_format_get_size (e->format)),
-                               d->new, sr);
-               }
-               break;
-       case EXIF_FORMAT_UNDEFINED:
-       case EXIF_FORMAT_BYTE:
-       case EXIF_FORMAT_ASCII:
-       default:
-               /* Nothing here. */
-               break;
-       }
+       exif_array_set_byte_order (e->format, e->data, e->components, d->old, d->new);
 }
 
 static void
@@ -958,4 +1011,29 @@ exif_data_set_byte_order (ExifData *data, ExifByteOrder order)
        d.new = order;
        exif_data_foreach_content (data, content_set_byte_order, &d);
        data->priv->order = order;
+       if (data->priv->md)
+               exif_mnote_data_set_byte_order (data->priv->md, order);
+}
+
+void
+exif_data_log (ExifData *data, ExifLog *log)
+{
+       unsigned int i;
+
+       if (!data || !data->priv) return;
+       exif_log_unref (data->priv->log);
+       data->priv->log = log;
+       exif_log_ref (log);
+
+       for (i = 0; i < EXIF_IFD_COUNT; i++)
+               exif_content_log (data->ifd[i], log);
+}
+
+/* Used internally within libexif */
+ExifLog *exif_data_get_log (ExifData *);
+ExifLog *
+exif_data_get_log (ExifData *data)
+{
+       if (!data || !data->priv) return NULL;
+       return data->priv->log;
 }