diff options
author | Matt Johnston <matt@ucc.asn.au> | 2007-02-03 13:31:01 +0000 |
---|---|---|
committer | Matt Johnston <matt@ucc.asn.au> | 2007-02-03 13:31:01 +0000 |
commit | d4bc0aec5d55a3df5fbb6f5a872d44c882ee71de (patch) | |
tree | 32980d182f7ec647ff91d4a6910f31c173245f6d /cli-authpubkey.c | |
parent | d0533106a8a5bfb81f1e95931db5a1a71ee11006 (diff) |
Fix leak of keybuf in recv_msg_userauth_pk_ok, courtesy of Klocwork
--HG--
extra : convert_revision : 9c39c3f447a47f61606df5d4bee364a449f12e18
Diffstat (limited to 'cli-authpubkey.c')
-rw-r--r-- | cli-authpubkey.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/cli-authpubkey.c b/cli-authpubkey.c index 9d36bc3..7ef25b4 100644 --- a/cli-authpubkey.c +++ b/cli-authpubkey.c @@ -60,8 +60,8 @@ void cli_pubkeyfail() { void recv_msg_userauth_pk_ok() { - struct SignKeyList *keyitem; - buffer* keybuf; + struct SignKeyList *keyitem = NULL; + buffer* keybuf = NULL; char* algotype = NULL; unsigned int algolen; int keytype; @@ -121,6 +121,8 @@ void recv_msg_userauth_pk_ok() { } else { TRACE(("That was whacky. We got told that a key was valid, but it didn't match our list. Sounds like dodgy code on Dropbear's part")) } + + buf_free(keybuf); TRACE(("leave recv_msg_userauth_pk_ok")) } |