diff options
author | David Shaw <[email protected]> | 2005-01-06 04:44:38 +0000 |
---|---|---|
committer | David Shaw <[email protected]> | 2005-01-06 04:44:38 +0000 |
commit | 299a250c944586dc5a6b48c77cbaf8cd8ab3ae95 (patch) | |
tree | 1d3e533414c3f646aa7f45b5fed4d59c4197ee7f /g10/armor.c | |
parent | * configure.ac: Use new GNUPG_CHECK_LIBUSB macro. (diff) | |
download | gnupg-299a250c944586dc5a6b48c77cbaf8cd8ab3ae95.tar.gz gnupg-299a250c944586dc5a6b48c77cbaf8cd8ab3ae95.zip |
* 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.
Diffstat (limited to 'g10/armor.c')
-rw-r--r-- | g10/armor.c | 25 |
1 files changed, 16 insertions, 9 deletions
diff --git a/g10/armor.c b/g10/armor.c index 1fe73f2ed..ca63f45f6 100644 --- a/g10/armor.c +++ b/g10/armor.c @@ -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; |