]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
ssh: fix scan-build warnings 867/head
authorVictor Julien <victor@inliniac.net>
Sun, 2 Mar 2014 10:57:13 +0000 (11:57 +0100)
committerVictor Julien <victor@inliniac.net>
Mon, 3 Mar 2014 16:34:57 +0000 (17:34 +0100)
app-layer-ssh.c:165:5: warning: Value stored to 'input_len' is never read
    input_len -= 1;
    ^            ~
1 warning generated.

app-layer-ssh.c:160:5: warning: Value stored to 'input_len' is never read
    input_len -= 4;
    ^            ~
1 warning generated.

src/app-layer-ssh.c

index 5a11ebb295e100274be32f80b3e0d702e39c2f3f..5c1106d4ba80378c1d691b227bffdd5ec1f7bb75 100644 (file)
@@ -140,6 +140,9 @@ static int SSHParseRecordHeader(SshState *state, SshHeader *header,
 {
 #ifdef DEBUG
     BUG_ON(input_len != 6);
+#else
+    if (input_len < 6)
+        SCReturnInt(-1);
 #endif
     /* input and input_len now point past initial line */
     uint32_t pkt_len = 0;
@@ -157,12 +160,12 @@ static int SSHParseRecordHeader(SshState *state, SshHeader *header,
     SCLogDebug("pkt len: %"PRIu32, pkt_len);
 
     input += 4;
-    input_len -= 4;
+    //input_len -= 4;
 
     header->padding_len = *input;
 
     input += 1;
-    input_len -= 1;
+    //input_len -= 1;
 
     SCLogDebug("padding: %u", header->padding_len);