From 16d62f6d94a449d04120ae35eb42796fe5b8bc57 Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Sun, 14 Jul 2013 10:46:55 +0000 Subject: [PATCH] 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.) [originally from svn r9926] --- sshpubk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 */