diff options
author | dtucker <dtucker> | 2013-06-05 22:22:05 +0000 |
---|---|---|
committer | dtucker <dtucker> | 2013-06-05 22:22:05 +0000 |
commit | 2322500c389d375bddfd2e6e163ed0a21656b94f (patch) | |
tree | 85b99964ab05e401d7258b83db4c608536e6e1a3 | |
parent | 9e79cbe4e73b63b54ea6c562dd7fbdb258fa377c (diff) | |
download | openssh-2322500c389d375bddfd2e6e163ed0a21656b94f.tar.gz |
- dtucker@cvs.openbsd.org 2013/06/05 12:52:38
[sshconnect2.c]
Fix memory leaks found by Zhenbo Xu and the Melton tool. bz#1967, ok djm
-rw-r--r-- | ChangeLog | 3 | ||||
-rw-r--r-- | sshconnect2.c | 8 |
2 files changed, 9 insertions, 2 deletions
@@ -40,6 +40,9 @@ When running sshd -D, close stderr unless we have explicitly requesting logging to stderr. From james.hunt at ubuntu.com via bz#1976, djm's patch so, err, ok dtucker. + - dtucker@cvs.openbsd.org 2013/06/05 12:52:38 + [sshconnect2.c] + Fix memory leaks found by Zhenbo Xu and the Melton tool. bz#1967, ok djm 20130602 - (tim) [Makefile.in] Make Solaris, UnixWare, & OpenServer linkers happy diff --git a/sshconnect2.c b/sshconnect2.c index 19ed3459..70e3cd8c 100644 --- a/sshconnect2.c +++ b/sshconnect2.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sshconnect2.c,v 1.197 2013/05/17 00:13:14 djm Exp $ */ +/* $OpenBSD: sshconnect2.c,v 1.198 2013/06/05 12:52:38 dtucker Exp $ */ /* * Copyright (c) 2000 Markus Friedl. All rights reserved. * Copyright (c) 2008 Damien Miller. All rights reserved. @@ -1322,8 +1322,11 @@ load_identity_file(char *filename, int userprovided) return NULL; } private = key_load_private_type(KEY_UNSPEC, filename, "", NULL, &perm_ok); - if (!perm_ok) + if (!perm_ok) { + if (private != NULL) + key_free(private); return NULL; + } if (private == NULL) { if (options.batch_mode) return NULL; @@ -1915,6 +1918,7 @@ authmethod_get(char *authlist) free(name); return current; } + free(name); } } |