From: simon Date: Sun, 14 Jul 2013 10:46:55 +0000 (+0000) Subject: Remove a redundant while-loop condition when reading RFC822-style X-Git-Url: https://git.distorted.org.uk/u/mdw/putty/commitdiff_plain/1ea566c3e96413d7edb40c50b0f7c7f8a5536079 Remove a redundant while-loop condition when reading RFC822-style header text from a PuTTY key file. (It's silly to have both while (len > 0) at the top of the loop _and_ an if (len == 0) return in the middle, and in fact the former was the erroneous one since it would have prohibited a 39-character header, which I intended to be permitted.) git-svn-id: svn://svn.tartarus.org/sgt/putty@9926 cda61777-01e9-0310-a592-d414129be87e --- diff --git a/sshpubk.c b/sshpubk.c index e5157a3c..ac9e0fa7 100644 --- a/sshpubk.c +++ b/sshpubk.c @@ -463,7 +463,7 @@ static int read_header(FILE * fp, char *header) int len = 39; int c; - while (len > 0) { + while (1) { c = fgetc(fp); if (c == '\n' || c == '\r' || c == EOF) return 0; /* failure */