Semantic fix in the X11 `authentication failed' error packet
authorsimon <simon@cda61777-01e9-0310-a592-d414129be87e>
Tue, 28 Aug 2001 12:24:50 +0000 (12:24 +0000)
committersimon <simon@cda61777-01e9-0310-a592-d414129be87e>
Tue, 28 Aug 2001 12:24:50 +0000 (12:24 +0000)
construction. Doesn't actually affect anything right now, since the
bug was a failure to round a length up to the next multiple of 4 and
it so happens that our current message was exactly 40 bytes anyway
:-) But if we start giving a wider variety of messages one day then
it might be handy to be able to do them without gratuitous crashes.

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

x11fwd.c

index 0e0df20..ac0ff04 100644 (file)
--- a/x11fwd.c
+++ b/x11fwd.c
@@ -308,7 +308,7 @@ int x11_send(Socket s, char *data, int len)
            reply[0] = 0;              /* failure */
            reply[1] = msglen;         /* length of reason string */
            memcpy(reply + 2, pr->firstpkt + 2, 4);     /* major/minor proto vsn */
-           PUT_16BIT(pr->firstpkt[0], reply + 6, msglen >> 2); /* data len */
+           PUT_16BIT(pr->firstpkt[0], reply + 6, msgsize >> 2);/* data len */
            memset(reply + 8, 0, msgsize);
            memcpy(reply + 8, message, msglen);
            sshfwd_write(pr->c, reply, 8 + msgsize);