Ubuntu
gthumb package

  1. Bug #316017
  2. Comment #10

Comment 10 for bug 316017

Revision history for this message
mjc (mjc-avtechpulse) wrote :

Tormod, your patch adds this bound-checking code:

if (i<tiff || i>readsize) return PATCH_EXIF_NO_TIFF;

Comments:

1) I don't think it is possible for (i < tiff) to ever occur in the code. Is it?

2) I think "i>readsize" should actually be "i>=readsize". Right?

- Mike