Re: [Libexif-devel] libexif/libexif exif-data.c,1.96,1.97

[Jan P. <pa...@pi...>]

> file: /cvsroot/libexif/libexif/libexif/exif-data.c,v
> -	if ((ifd < 0) || (ifd >= EXIF_IFD_COUNT))
> -		return;
> +	/* check for valid ExifIfd enum range
> +	 * if ((((int)ifd) < 0) || (ifd >= EXIF_IFD_COUNT))
> +	 * return;
> +	*/
> +	switch (ifd) {
> +	case EXIF_IFD_0:
> +	case EXIF_IFD_1:
> +	case EXIF_IFD_EXIF:
> +	case EXIF_IFD_GPS:
> +	case EXIF_IFD_INTEROPERABILITY:
> +	case EXIF_IFD_COUNT:
> +	  break;

Hans,

EXIF_IFD_COUNT is not a valid IFD ID - it is the counter of the IDs.

 -- Jan