}
}
+static void
+clear_entry (ExifEntry *e)
+{
+ e->components = 0;
+ e->size = 0;
+}
+
/*! Get a value and convert it to an ExifShort.
* \bug Not all types are converted that could be converted and no indication
* is made when that occurs
if (e->size < 8) {
e->data = exif_entry_realloc (e, e->data, 8 + e->size);
if (!e->data) {
- e->size = 0;
- e->components = 0;
+ clear_entry(e);
return;
}
memcmp (e->data, "\0\0\0\0\0\0\0\0", 8)) {
e->data = exif_entry_realloc (e, e->data, 8 + e->size);
if (!e->data) {
- e->size = 0;
- e->components = 0;
+ clear_entry(e);
break;
}
}
/*! Format the value of an ExifEntry for human display in a generic way.
- * The output is localized. The formatting is independent of the tag number.
- * \pre The buffer at val is entirely cleared to 0. This guarantees that the
- * resulting string will be NUL terminated.
+ * The output is localized. The formatting is independent of the tag number
+ * and is based entirely on the data type.
* \pre The ExifEntry is already a member of an ExifData.
* \param[in] e EXIF entry
* \param[out] val buffer in which to store value
- * \param[in] maxlen one less than the length of the buffer val
+ * \param[in] maxlen the length of the buffer val
*/
static void
exif_entry_format_value(ExifEntry *e, char *val, size_t maxlen)
ExifRational v_rat;
ExifSRational v_srat;
ExifSLong v_slong;
- char b[64];
unsigned int i;
+ size_t len;
const ExifByteOrder o = exif_data_get_byte_order (e->parent->parent);
- if (!e->size)
+ if (!e->size || !maxlen)
return;
switch (e->format) {
case EXIF_FORMAT_UNDEFINED:
case EXIF_FORMAT_SBYTE:
v_byte = e->data[0];
snprintf (val, maxlen, "0x%02x", v_byte);
- maxlen -= strlen (val);
+ len = strlen (val);
for (i = 1; i < e->components; i++) {
v_byte = e->data[i];
- snprintf (b, sizeof (b), ", 0x%02x", v_byte);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed)maxlen <= 0) break;
+ snprintf (val+len, maxlen-len, ", 0x%02x", v_byte);
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_SHORT:
v_short = exif_get_short (e->data, o);
snprintf (val, maxlen, "%u", v_short);
- maxlen -= strlen (val);
+ len = strlen (val);
for (i = 1; i < e->components; i++) {
v_short = exif_get_short (e->data +
exif_format_get_size (e->format) * i, o);
- snprintf (b, sizeof (b), ", %u", v_short);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed)maxlen <= 0) break;
+ snprintf (val+len, maxlen-len, ", %u", v_short);
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_SSHORT:
v_sshort = exif_get_sshort (e->data, o);
snprintf (val, maxlen, "%i", v_sshort);
- maxlen -= strlen (val);
+ len = strlen (val);
for (i = 1; i < e->components; i++) {
v_sshort = exif_get_short (e->data +
exif_format_get_size (e->format) *
i, o);
- snprintf (b, sizeof (b), ", %i", v_sshort);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed)maxlen <= 0) break;
+ snprintf (val+len, maxlen-len, ", %i", v_sshort);
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_LONG:
v_long = exif_get_long (e->data, o);
snprintf (val, maxlen, "%lu", (unsigned long) v_long);
- maxlen -= strlen (val);
+ len = strlen (val);
for (i = 1; i < e->components; i++) {
v_long = exif_get_long (e->data +
exif_format_get_size (e->format) *
i, o);
- snprintf (b, sizeof (b), ", %lu", (unsigned long) v_long);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed)maxlen <= 0) break;
+ snprintf (val+len, maxlen-len, ", %lu", (unsigned long) v_long);
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_SLONG:
v_slong = exif_get_slong (e->data, o);
snprintf (val, maxlen, "%li", (long) v_slong);
- maxlen -= strlen (val);
+ len = strlen (val);
for (i = 1; i < e->components; i++) {
v_slong = exif_get_slong (e->data +
exif_format_get_size (e->format) * i, o);
- snprintf (b, sizeof (b), ", %li", (long) v_slong);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed)maxlen <= 0) break;
+ snprintf (val+len, maxlen-len, ", %li", (long) v_slong);
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_ASCII:
- strncpy (val, (char *) e->data, MIN (maxlen, e->size));
+ strncpy (val, (char *) e->data, MIN (maxlen-1, e->size));
+ val[MIN (maxlen-1, e->size)] = 0;
break;
case EXIF_FORMAT_RATIONAL:
+ len = 0;
for (i = 0; i < e->components; i++) {
if (i > 0) {
- strncat (val, ", ", maxlen);
- maxlen -= 2;
+ snprintf (val+len, maxlen-len, ", ");
+ len += strlen (val+len);
}
v_rat = exif_get_rational (
e->data + 8 * i, o);
* range 13..120 will show 2 decimal points.
*/
int decimals = (int)(log10(v_rat.denominator)-0.08+1.0);
- snprintf (b, sizeof (b), "%2.*f",
+ snprintf (val+len, maxlen-len, "%2.*f",
decimals,
(double) v_rat.numerator /
(double) v_rat.denominator);
} else
- snprintf (b, sizeof (b), "%lu/%lu",
+ snprintf (val+len, maxlen-len, "%lu/%lu",
(unsigned long) v_rat.numerator,
(unsigned long) v_rat.denominator);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed) maxlen <= 0) break;
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_SRATIONAL:
+ len = 0;
for (i = 0; i < e->components; i++) {
if (i > 0) {
- strncat (val, ", ", maxlen);
- maxlen -= 2;
+ snprintf (val+len, maxlen-len, ", ");
+ len += strlen (val+len);
}
v_srat = exif_get_srational (
e->data + 8 * i, o);
if (v_srat.denominator) {
int decimals = (int)(log10(fabs(v_srat.denominator))-0.08+1.0);
- snprintf (b, sizeof (b), "%2.*f",
+ snprintf (val+len, maxlen-len, "%2.*f",
decimals,
(double) v_srat.numerator /
(double) v_srat.denominator);
} else
- snprintf (b, sizeof (b), "%li/%li",
+ snprintf (val+len, maxlen-len, "%li/%li",
(long) v_srat.numerator,
(long) v_srat.denominator);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed) maxlen <= 0) break;
+ len += strlen (val+len);
+ if (len >= maxlen-1) break;
}
break;
case EXIF_FORMAT_DOUBLE:
printf ("%s Value: %s\n", buf, exif_entry_get_value (e, value, sizeof(value)));
}
+/*! Check if a string consists entirely of a single, repeated character.
+ * Up to first n bytes are checked.
+ *
+ * \param[in] data pointer of string to check
+ * \param[in] ch character to match
+ * \param[in] n maximum number of characters to match
+ *
+ * \return 0 if the string matches or is of zero length, nonzero otherwise
+ */
+static int
+match_repeated_char(const unsigned char *data, unsigned char ch, size_t n)
+{
+ int i;
+ for (i=n; i; --i, ++data) {
+ if (*data == 0) {
+ i = 0; /* all bytes before NUL matched */
+ break;
+ }
+ if (*data != ch)
+ break;
+ }
+ return i;
+}
+
#define CF(entry,target,v,maxlen) \
{ \
if (entry->format != target) { \
exif_entry_get_value (ExifEntry *e, char *val, unsigned int maxlen)
{
unsigned int i, j, k;
- const unsigned char *t;
ExifShort v_short, v_short2, v_short3, v_short4;
ExifByte v_byte;
ExifRational v_rat;
{"0210", 2, 1},
{"0220", 2, 2},
{"0221", 2, 21},
+ {"0230", 2, 3},
{"" , 0, 0}
};
- /* FIXME: This belongs to somewhere else. */
- /* libexif should use the default system locale.
- * If an application specifically requires UTF-8, then we
- * must give the application a way to tell libexif that.
- *
- * bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8");
- */
- bindtextdomain (GETTEXT_PACKAGE, LOCALEDIR);
+ (void) bindtextdomain (GETTEXT_PACKAGE, LOCALEDIR);
+
+ if (!e || !e->parent || !e->parent->parent || !maxlen || !val)
+ return val;
/* make sure the returned string is zero terminated */
+ /* FIXME: this is inefficient in the case of long buffers and should
+ * instead be taken care of on each write instead. */
memset (val, 0, maxlen);
- maxlen--;
- memset (b, 0, sizeof (b));
/* We need the byte order */
- if (!e || !e->parent || !e->parent->parent)
- return val;
o = exif_data_get_byte_order (e->parent->parent);
/* Sanity check */
* NULL terminated.
*/
if ((e->size >= 8) && !memcmp (e->data, "ASCII\0\0\0", 8)) {
- strncpy (val, (char *) e->data + 8, MIN (e->size - 8, maxlen));
+ strncpy (val, (char *) e->data + 8, MIN (e->size - 8, maxlen-1));
break;
}
if ((e->size >= 8) && !memcmp (e->data, "UNICODE\0", 8)) {
- strncpy (val, _("Unsupported UNICODE string"), maxlen);
+ strncpy (val, _("Unsupported UNICODE string"), maxlen-1);
/* FIXME: use iconv to convert into the locale encoding.
* EXIF 2.2 implies (but does not say) that this encoding is
* UCS-2.
break;
}
if ((e->size >= 8) && !memcmp (e->data, "JIS\0\0\0\0\0", 8)) {
- strncpy (val, _("Unsupported JIS string"), maxlen);
+ strncpy (val, _("Unsupported JIS string"), maxlen-1);
/* FIXME: use iconv to convert into the locale encoding */
break;
}
/*
* If we reach this point, the tag does not
- * comply with the standard and seems to contain data.
+ * comply with the standard but seems to contain data.
* Print as much as possible.
+ * Note: make sure we do not overwrite the final \0 at maxlen-1
*/
exif_entry_log (e, EXIF_LOG_CODE_DEBUG,
- _("Tag UserComment does not comply "
- "with standard but contains data."));
- for (; (i < e->size) && (strlen (val) < maxlen - 1); i++) {
+ _("Tag UserComment contains data but is "
+ "against specification."));
+ for (j = 0; (i < e->size) && (j < maxlen-1); i++, j++) {
exif_entry_log (e, EXIF_LOG_CODE_DEBUG,
_("Byte at position %i: 0x%02x"), i, e->data[i]);
- val[strlen (val)] =
- isprint (e->data[i]) ? e->data[i] : '.';
+ val[j] = isprint (e->data[i]) ? e->data[i] : '.';
}
break;
case EXIF_TAG_EXIF_VERSION:
CF (e, EXIF_FORMAT_UNDEFINED, val, maxlen);
CC (e, 4, val, maxlen);
- strncpy (val, _("Unknown Exif Version"), maxlen);
+ strncpy (val, _("Unknown Exif Version"), maxlen-1);
for (i = 0; *versions[i].label; i++) {
if (!memcmp (e->data, versions[i].label, 4)) {
snprintf (val, maxlen,
CF (e, EXIF_FORMAT_UNDEFINED, val, maxlen);
CC (e, 4, val, maxlen);
if (!memcmp (e->data, "0100", 4))
- strncpy (val, _("FlashPix Version 1.0"), maxlen);
+ strncpy (val, _("FlashPix Version 1.0"), maxlen-1);
else if (!memcmp (e->data, "0101", 4))
- strncpy (val, _("FlashPix Version 1.01"), maxlen);
+ strncpy (val, _("FlashPix Version 1.01"), maxlen-1);
else
- strncpy (val, _("Unknown FlashPix Version"), maxlen);
+ strncpy (val, _("Unknown FlashPix Version"), maxlen-1);
break;
case EXIF_TAG_COPYRIGHT:
CF (e, EXIF_FORMAT_ASCII, val, maxlen);
/*
* First part: Photographer.
* Some cameras store a string like " " here. Ignore it.
+ * Remember that a corrupted tag might not be NUL-terminated
*/
- if (e->size && e->data &&
- (strspn ((char *)e->data, " ") != strlen ((char *) e->data)))
- strncpy (val, (char *) e->data, MIN (maxlen, e->size));
+ if (e->size && e->data && match_repeated_char(e->data, ' ', e->size))
+ strncpy (val, (char *) e->data, MIN (maxlen-1, e->size));
else
- strncpy (val, _("[None]"), maxlen);
- strncat (val, " ", maxlen - strlen (val));
- strncat (val, _("(Photographer)"), maxlen - strlen (val));
+ strncpy (val, _("[None]"), maxlen-1);
+ strncat (val, " ", maxlen-1 - strlen (val));
+ strncat (val, _("(Photographer)"), maxlen-1 - strlen (val));
/* Second part: Editor. */
- strncat (val, " - ", maxlen - strlen (val));
+ strncat (val, " - ", maxlen-1 - strlen (val));
+ k = 0;
if (e->size && e->data) {
- size_t ts;
- t = e->data + strlen ((char *) e->data) + 1;
- ts = e->data + e->size - t;
- if ((ts > 0) && (strspn ((char *)t, " ") != ts))
- strncat (val, (char *)t, MIN (maxlen - strlen (val), ts));
- } else {
- strncat (val, _("[None]"), maxlen - strlen (val));
+ const unsigned char *tagdata = memchr(e->data, 0, e->size);
+ if (tagdata++) {
+ unsigned int editor_ofs = tagdata - e->data;
+ unsigned int remaining = e->size - editor_ofs;
+ if (match_repeated_char(tagdata, ' ', remaining)) {
+ strncat (val, (const char*)tagdata, MIN (maxlen-1 - strlen (val), remaining));
+ ++k;
+ }
+ }
}
- strncat (val, " ", maxlen - strlen (val));
- strncat (val, _("(Editor)"), maxlen - strlen (val));
+ if (!k)
+ strncat (val, _("[None]"), maxlen-1 - strlen (val));
+ strncat (val, " ", maxlen-1 - strlen (val));
+ strncat (val, _("(Editor)"), maxlen-1 - strlen (val));
break;
case EXIF_TAG_FNUMBER:
d = (double) v_rat.numerator / (double) v_rat.denominator;
snprintf (val, maxlen, _("%.02f EV"), d);
snprintf (b, sizeof (b), _(" (f/%.01f)"), pow (2, d / 2.));
- if (maxlen > strlen (val) + strlen (b))
- strncat (val, b, maxlen - strlen (val));
+ strncat (val, b, maxlen-1 - strlen (val));
break;
case EXIF_TAG_FOCAL_LENGTH:
CF (e, EXIF_FORMAT_RATIONAL, val, maxlen);
}
}
if (d)
- snprintf (b, sizeof (b), _(" (35 equivalent: %d mm)"),
- (int) (d * (double) v_rat.numerator /
- (double) v_rat.denominator));
+ snprintf (b, sizeof (b), _(" (35 equivalent: %.0f mm)"),
+ (d * (double) v_rat.numerator /
+ (double) v_rat.denominator));
+ else
+ b[0] = 0;
d = (double) v_rat.numerator / (double) v_rat.denominator;
snprintf (val, maxlen, "%.1f mm", d);
- if (maxlen > strlen (val) + strlen (b))
- strncat (val, b, maxlen - strlen (val));
+ strncat (val, b, maxlen-1 - strlen (val));
break;
case EXIF_TAG_SUBJECT_DISTANCE:
CF (e, EXIF_FORMAT_RATIONAL, val, maxlen);
}
d = (double) v_rat.numerator / (double) v_rat.denominator;
if (d < 1)
- snprintf (val, maxlen, _("1/%i"), (int) (0.5 + 1. / d));
+ snprintf (val, maxlen, _("1/%.0f"), 1. / d);
else
- snprintf (val, maxlen, "%i", (int) d);
- if (maxlen > strlen (val) + strlen (_(" sec.")))
- strncat (val, _(" sec."), maxlen - strlen (val));
+ snprintf (val, maxlen, "%.0f", d);
+ strncat (val, _(" sec."), maxlen-1 - strlen (val));
break;
case EXIF_TAG_SHUTTER_SPEED_VALUE:
CF (e, EXIF_FORMAT_SRATIONAL, val, maxlen);
snprintf (val, maxlen, _("%.02f EV"), d);
d = 1. / pow (2, d);
if (d < 1)
- snprintf (b, sizeof (b), _(" (1/%d sec.)"), (int) (1. / d));
+ snprintf (b, sizeof (b), _(" (1/%.0f sec.)"), 1. / d);
else
- snprintf (b, sizeof (b), _(" (%d sec.)"), (int) d);
- strncat (val, b, maxlen - strlen (val));
+ snprintf (b, sizeof (b), _(" (%.0f sec.)"), d);
+ strncat (val, b, maxlen-1 - strlen (val));
break;
case EXIF_TAG_BRIGHTNESS_VALUE:
CF (e, EXIF_FORMAT_SRATIONAL, val, maxlen);
snprintf (val, maxlen, _("%.02f EV"), d);
snprintf (b, sizeof (b), _(" (%.02f cd/m^2)"),
1. / (M_PI * 0.3048 * 0.3048) * pow (2, d));
- if (maxlen > strlen (val) + strlen (b))
- strncat (val, b, maxlen - strlen (val));
+ strncat (val, b, maxlen-1 - strlen (val));
break;
case EXIF_TAG_FILE_SOURCE:
CF (e, EXIF_FORMAT_UNDEFINED, val, maxlen);
CC (e, 1, val, maxlen);
v_byte = e->data[0];
if (v_byte == 3)
- strncpy (val, _("DSC"), maxlen);
+ strncpy (val, _("DSC"), maxlen-1);
else
snprintf (val, maxlen, _("Internal error (unknown "
"value %i)"), v_byte);
case 6: c = _("B"); break;
default: c = _("Reserved"); break;
}
- strncat (val, c, maxlen - strlen (val));
+ strncat (val, c, maxlen-1 - strlen (val));
if (i < 3)
- strncat (val, " ", maxlen - strlen (val));
+ strncat (val, " ", maxlen-1 - strlen (val));
}
break;
case EXIF_TAG_EXPOSURE_BIAS_VALUE:
CC (e, 1, val, maxlen);
v_byte = e->data[0];
if (v_byte == 1)
- strncpy (val, _("Directly photographed"), maxlen);
+ strncpy (val, _("Directly photographed"), maxlen-1);
else
snprintf (val, maxlen, _("Internal error (unknown "
"value %i)"), v_byte);
e->data + exif_format_get_size (e->format),
o);
if ((v_short == 2) && (v_short2 == 1))
- strncpy (val, _("YCbCr4:2:2"), maxlen);
+ strncpy (val, _("YCbCr4:2:2"), maxlen-1);
else if ((v_short == 2) && (v_short2 == 2))
- strncpy (val, _("YCbCr4:2:0"), maxlen);
+ strncpy (val, _("YCbCr4:2:0"), maxlen-1);
else
snprintf (val, maxlen, "%u, %u", v_short, v_short2);
break;
CC (e, 4, val, maxlen);
v_byte = e->data[0];
snprintf (val, maxlen, "%u", v_byte);
- maxlen -= strlen (val);
for (i = 1; i < e->components; i++) {
v_byte = e->data[i];
snprintf (b, sizeof (b), ".%u", v_byte);
- strncat (val, b, maxlen);
- maxlen -= strlen (b);
- if ((signed)maxlen <= 0) break;
+ strncat (val, b, maxlen-1 - strlen (val));
}
break;
case EXIF_TAG_INTEROPERABILITY_VERSION:
/* a.k.a. case EXIF_TAG_GPS_LATITUDE: */
/* This tag occurs in EXIF_IFD_INTEROPERABILITY */
if (e->format == EXIF_FORMAT_UNDEFINED) {
- strncpy (val, (char *) e->data, MIN (maxlen, e->size));
+ strncpy (val, (char *) e->data, MIN (maxlen-1, e->size));
break;
}
/* EXIF_TAG_GPS_LATITUDE is the same numerically as
CC (e, 1, val, maxlen);
v_byte = e->data[0];
if (v_byte == 0)
- strncpy (val, _("Sea level"), maxlen);
+ strncpy (val, _("Sea level"), maxlen-1);
else if (v_byte == 1)
- strncpy (val, _("Sea level reference"), maxlen);
+ strncpy (val, _("Sea level reference"), maxlen-1);
else
snprintf (val, maxlen, _("Internal error (unknown "
"value %i)"), v_byte);
for (k = 0; list2[i].elem[j].values[k]; k++) {
size_t l = strlen (_(list2[i].elem[j].values[k]));
if ((maxlen > l) && (strlen (val) < l))
- strncpy (val, _(list2[i].elem[j].values[k]), maxlen);
+ strncpy (val, _(list2[i].elem[j].values[k]), maxlen-1);
}
if (!val[0]) snprintf (val, maxlen, "%i", v_short);
else if (!*list[i].strings[j])
snprintf (val, maxlen, _("Unknown value %i"), v_short);
else
- strncpy (val, _(list[i].strings[j]), maxlen);
+ strncpy (val, _(list[i].strings[j]), maxlen-1);
break;
case EXIF_TAG_XP_TITLE:
case EXIF_TAG_XP_AUTHOR:
case EXIF_TAG_XP_KEYWORDS:
case EXIF_TAG_XP_SUBJECT:
+ {
+ unsigned short *utf16;
+
+ /* Sanity check the size to prevent overflow */
+ if (e->size+sizeof(unsigned short) < e->size) break;
+
+ /* The tag may not be U+0000-terminated , so make a local
+ U+0000-terminated copy before converting it */
+ utf16 = exif_mem_alloc (e->priv->mem, e->size+sizeof(unsigned short));
+ if (!utf16) break;
+ memcpy(utf16, e->data, e->size);
+
+ /* NUL terminate the string. If the size is odd (which isn't possible
+ * for a UTF16 string), then this will overwrite the final garbage byte.
+ */
+ utf16[e->size/sizeof(unsigned short)] = 0;
+
/* Warning! The texts are converted from UTF16 to UTF8 */
/* FIXME: use iconv to convert into the locale encoding */
- exif_convert_utf16_to_utf8(val, (unsigned short*)e->data, MIN(maxlen, e->size));
+ exif_convert_utf16_to_utf8(val, utf16, maxlen);
+ exif_mem_free(e->priv->mem, utf16);
break;
+ }
default:
/* Use a generic value formatting */
e->format = EXIF_FORMAT_LONG;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
break;
/* SHORT, 1 component, no default */
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 0);
break;
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 1);
break;
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 2);
break;
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 3);
break;
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 0xffff);
break;
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 8);
exif_set_short (
e->data + exif_format_get_size (e->format),
e->format = EXIF_FORMAT_SHORT;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
exif_set_short (e->data, o, 2);
exif_set_short (
e->data + exif_format_get_size (e->format),
e->format = EXIF_FORMAT_SRATIONAL;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
break;
/* RATIONAL, 1 component, no default */
e->format = EXIF_FORMAT_RATIONAL;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
break;
/* RATIONAL, 1 component, default 72/1 */
e->format = EXIF_FORMAT_RATIONAL;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
r.numerator = 72;
r.denominator = 1;
exif_set_rational (e->data, o, r);
e->format = EXIF_FORMAT_RATIONAL;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
break;
/* RATIONAL, 6 components */
e->format = EXIF_FORMAT_RATIONAL;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
r.denominator = 1;
r.numerator = 0;
exif_set_rational (e->data, o, r);
e->format = EXIF_FORMAT_ASCII;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
snprintf ((char *) e->data, e->size,
"%04i:%02i:%02i %02i:%02i:%02i",
tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
e->format = EXIF_FORMAT_ASCII;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
strncpy ((char *)e->data, _("[None]"), e->size);
break;
/* ASCII, default "[None]\0[None]\0" */
e->format = EXIF_FORMAT_ASCII;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
strcpy (((char *)e->data) + 0, _("[None]"));
strcpy (((char *)e->data) + strlen (_("[None]")) + 1, _("[None]"));
break;
e->format = EXIF_FORMAT_UNDEFINED;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
e->data[0] = 0x01;
break;
e->format = EXIF_FORMAT_UNDEFINED;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
e->data[0] = 0x03;
break;
e->format = EXIF_FORMAT_UNDEFINED;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
memcpy (e->data, "0100", 4);
break;
e->format = EXIF_FORMAT_UNDEFINED;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
memcpy (e->data, "0210", 4);
break;
e->format = EXIF_FORMAT_UNDEFINED;
e->size = exif_format_get_size (e->format) * e->components;
e->data = exif_entry_alloc (e, e->size);
- if (!e->data) break;
+ if (!e->data) { clear_entry(e); break; }
e->data[0] = 1;
e->data[1] = 2;
e->data[2] = 3;