diff options
author | Darren Tucker <dtucker@zip.com.au> | 2012-12-07 13:07:02 +1100 |
---|---|---|
committer | Darren Tucker <dtucker@zip.com.au> | 2012-12-07 13:07:02 +1100 |
commit | 8a96522482acd40af2e8a08696780a54c00b4feb (patch) | |
tree | 94fb15f82793c6c518852475235ee8a2dfda82fb /ssh-add.c | |
parent | f9333d5246d979a448c5ff4d466de2fd2d286cfd (diff) |
- markus@cvs.openbsd.org 2012/12/05 15:42:52
[ssh-add.c]
prevent double-free of comment; ok djm@
Diffstat (limited to 'ssh-add.c')
-rw-r--r-- | ssh-add.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: ssh-add.c,v 1.104 2012/12/02 20:42:15 djm Exp $ */ | 1 | /* $OpenBSD: ssh-add.c,v 1.105 2012/12/05 15:42:52 markus Exp $ */ |
2 | /* | 2 | /* |
3 | * Author: Tatu Ylonen <ylo@cs.hut.fi> | 3 | * Author: Tatu Ylonen <ylo@cs.hut.fi> |
4 | * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland | 4 | * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland |
@@ -118,6 +118,7 @@ delete_file(AuthenticationConnection *ac, const char *filename, int key_only) | |||
118 | 118 | ||
119 | /* Now try to delete the corresponding certificate too */ | 119 | /* Now try to delete the corresponding certificate too */ |
120 | free(comment); | 120 | free(comment); |
121 | comment = NULL; | ||
121 | xasprintf(&certpath, "%s-cert.pub", filename); | 122 | xasprintf(&certpath, "%s-cert.pub", filename); |
122 | if ((cert = key_load_public(certpath, &comment)) == NULL) | 123 | if ((cert = key_load_public(certpath, &comment)) == NULL) |
123 | goto out; | 124 | goto out; |