Another valgrind-caught error. This one has apparently been there

since the Dawn O' Time, and consisted of me putting the two halves
of a short-circuiting bounds check the wrong way round: instead of
`p_in_range && *p', I had `*p && p_in_range'. Oops. valgrind rocks.

git-svn-id: svn://svn.tartarus.org/sgt/putty@2174 cda61777-01e9-0310-a592-d414129be87e

diff --git a/telnet.c b/telnet.c
index f20488e..7e612cf 100644 (file)
--- a/telnet.c
+++ b/telnet.c
@@ -742,7 +742,7 @@ static int telnet_send(void *handle, char *buf, int len)
     while (p < buf + len) {
        char *q = p;
 
-       while (iswritable((unsigned char) *p) && p < buf + len)
+       while (p < buf + len && iswritable((unsigned char) *p))
            p++;
        telnet->bufsize = sk_write(telnet->s, q, p - q);