* photoid.c (generate_photo_id): Check for the JPEG magic numbers
authorDavid Shaw <dshaw@jabberwocky.com>
Wed, 6 Apr 2011 03:47:58 +0000 (23:47 -0400)
committerDavid Shaw <dshaw@jabberwocky.com>
Wed, 6 Apr 2011 03:47:58 +0000 (23:47 -0400)
instead of JFIF since some programs generate an EXIF header first.
This is issue 1331.

g10/ChangeLog
g10/photoid.c

index 8479663..19db8a4 100644 (file)
@@ -1,3 +1,9 @@
+2011-04-05  David Shaw  <dshaw@jabberwocky.com>
+
+       * photoid.c (generate_photo_id): Check for the JPEG magic numbers
+       instead of JFIF since some programs generate an EXIF header first.
+       This is issue 1331.
+
 2011-02-23  Werner Koch  <wk@g10code.com>
 
        * Makefile.am (LDADD): Move LIBREADLINE to ..
index 727a9ff..30c8f7f 100644 (file)
@@ -1,5 +1,5 @@
 /* photoid.c - photo ID handling code
- * Copyright (C) 2001, 2002, 2005, 2006, 2008 Free Software Foundation, Inc.
+ * Copyright (C) 2001, 2002, 2005, 2006, 2008, 2011 Free Software Foundation, Inc.
  *
  * This file is part of GnuPG.
  *
@@ -139,8 +139,7 @@ generate_photo_id(PKT_public_key *pk,const char *photo_name)
       iobuf_close(file);
 
       /* Is it a JPEG? */
-      if(photo[0]!=0xFF || photo[1]!=0xD8 ||
-        photo[6]!='J' || photo[7]!='F' || photo[8]!='I' || photo[9]!='F')
+      if(photo[0]!=0xFF || photo[1]!=0xD8)
        {
          log_error(_("`%s' is not a JPEG file\n"),filename);
          xfree(photo);