upstream: Don't leak SK device. Patch from Pedro Martelletto via

github PR#316. ok djm@

OpenBSD-Commit-ID: 17d11327545022e727d95fd08b213171c5a4585d
This commit is contained in:
dtucker@openbsd.org 2022-04-29 03:16:48 +00:00 committed by Damien Miller
parent 247082b501
commit 3e26b3a6ee
2 changed files with 4 additions and 2 deletions

View File

@ -1,4 +1,4 @@
/* $OpenBSD: sk-usbhid.c,v 1.38 2022/02/07 01:25:12 djm Exp $ */ /* $OpenBSD: sk-usbhid.c,v 1.39 2022/04/29 03:16:48 dtucker Exp $ */
/* /*
* Copyright (c) 2019 Markus Friedl * Copyright (c) 2019 Markus Friedl
* Copyright (c) 2020 Pedro Martelletto * Copyright (c) 2020 Pedro Martelletto
@ -1393,6 +1393,7 @@ sk_load_resident_keys(const char *pin, struct sk_option **options,
freezero(rks[i]->user_id, rks[i]->user_id_len); freezero(rks[i]->user_id, rks[i]->user_id_len);
freezero(rks[i], sizeof(*rks[i])); freezero(rks[i], sizeof(*rks[i]));
} }
free(device);
free(rks); free(rks);
return ret; return ret;
} }

View File

@ -1,4 +1,4 @@
/* $OpenBSD: ssh-sk-helper.c,v 1.12 2021/10/28 02:54:18 djm Exp $ */ /* $OpenBSD: ssh-sk-helper.c,v 1.13 2022/04/29 03:16:48 dtucker Exp $ */
/* /*
* Copyright (c) 2019 Google LLC * Copyright (c) 2019 Google LLC
* *
@ -265,6 +265,7 @@ process_load_resident(struct sshbuf *req)
sshsk_free_resident_keys(srks, nsrks); sshsk_free_resident_keys(srks, nsrks);
sshbuf_free(kbuf); sshbuf_free(kbuf);
free(provider); free(provider);
free(device);
if (pin != NULL) if (pin != NULL)
freezero(pin, strlen(pin)); freezero(pin, strlen(pin));
return resp; return resp;