* g10.c (main): Typo.
authorDavid Shaw <dshaw@jabberwocky.com>
Thu, 6 Jan 2005 04:44:38 +0000 (04:44 +0000)
committerDavid Shaw <dshaw@jabberwocky.com>
Thu, 6 Jan 2005 04:44:38 +0000 (04:44 +0000)
* armor.c (is_armor_header): Allow CR and LF (not just actual spaces) in
an armor header line (-----BEGIN etc).  This is needed due to CRLF issues
on win32.  As before, --openpgp makes it strict.

g10/ChangeLog
g10/armor.c
g10/g10.c

index 11639c9..8493bda 100644 (file)
@@ -1,3 +1,12 @@
+2005-01-05  David Shaw  <dshaw@jabberwocky.com>
+
+       * g10.c (main): Typo.
+
+       * armor.c (is_armor_header): Allow CR and LF (not just actual
+       spaces) in an armor header line (-----BEGIN etc).  This is needed
+       due to CRLF issues on win32.  As before, --openpgp makes it
+       strict.
+
 2005-01-03  David Shaw  <dshaw@jabberwocky.com>
 
        * Makefile.am: Use @LIBUSB@ instead of @LIBUSB_LIBS@
index 1fe73f2..ca63f45 100644 (file)
@@ -291,17 +291,24 @@ is_armor_header( byte *line, unsigned len )
     save_p = p;
     p += 5;
 
-    /* Some mail programs on Windows seem to add spaces to the end of
-       the line.  This becomes strict if --openpgp is set. */
-
-    if(!RFC2440)
-      while(*p==' ')
+    /* Some Windows environments seem to add whitespace to the end of
+       the line, so we strip it here.  This becomes strict if
+       --rfc2440 is set since 2440 reads "The header lines, therefore,
+       MUST start at the beginning of a line, and MUST NOT have text
+       following them on the same line."  It is unclear whether "text"
+       refers to all text or just non-whitespace text. */
+
+    if(RFC2440)
+      {
+       if( *p == '\r' )
+         p++;
+       if( *p == '\n' )
+         p++;
+      }
+    else
+      while(*p==' ' || *p=='\r' || *p=='\n')
        p++;
 
-    if( *p == '\r' )
-       p++;
-    if( *p == '\n' )
-       p++;
     if( *p )
        return -1; /* garbage after dashes */
     save_c = *save_p; *save_p = 0;
index 3919f68..3e773dc 100644 (file)
--- a/g10/g10.c
+++ b/g10/g10.c
@@ -2607,8 +2607,8 @@ main( int argc, char **argv )
 
     if(require_secmem && !got_secmem)
       {
-       log_info(_("will not run with insecure memory due to %s"),
-                "--require-secmem\n");
+       log_info(_("will not run with insecure memory due to %s\n"),
+                "--require-secmem");
        g10_exit(2);
       }