]> asedeno.scripts.mit.edu Git - PuTTY.git/commitdiff
Simplify ssh_pkt_addstring_str().
authorBen Harris <bjh21@bjh21.me.uk>
Sun, 10 May 2015 20:12:37 +0000 (21:12 +0100)
committerBen Harris <bjh21@bjh21.me.uk>
Mon, 11 May 2015 21:10:23 +0000 (22:10 +0100)
It's just ssh_pkt_addstring_data but using strlen to get the length of
string to add, so make that explicit by having it call
ssh_pkt_addstring_data.  Good compilers should be unaffected by this
change.

ssh.c

diff --git a/ssh.c b/ssh.c
index 14bff6fe2919e3f5ce3333d1098a9e01927d7afd..9feed64d38e78611c2c024d1834d2f71ac9d30be 100644 (file)
--- a/ssh.c
+++ b/ssh.c
@@ -2127,17 +2127,16 @@ static void ssh_pkt_addstring_start(struct Packet *pkt)
     ssh_pkt_adduint32(pkt, 0);
     pkt->savedpos = pkt->length;
 }
-static void ssh_pkt_addstring_str(struct Packet *pkt, const char *data)
-{
-    ssh_pkt_adddata(pkt, data, strlen(data));
-    PUT_32BIT(pkt->data + pkt->savedpos - 4, pkt->length - pkt->savedpos);
-}
 static void ssh_pkt_addstring_data(struct Packet *pkt, const char *data,
                                    int len)
 {
     ssh_pkt_adddata(pkt, data, len);
     PUT_32BIT(pkt->data + pkt->savedpos - 4, pkt->length - pkt->savedpos);
 }
+static void ssh_pkt_addstring_str(struct Packet *pkt, const char *data)
+{
+  ssh_pkt_addstring_data(pkt, data, strlen(data));
+}
 static void ssh_pkt_addstring(struct Packet *pkt, const char *data)
 {
     ssh_pkt_addstring_start(pkt);