diff options
| author |   angelos <angelos@openbsd.org> | 2002-06-10 21:09:27 +0000 |
|---|---|---|
| committer | angelos <angelos@openbsd.org> | 2002-06-10 21:09:27 +0000 |
| commit | 121c0ac5c5f314dbfb18379b899a72365f7c7e15 (patch) | |
| tree | cecd31f3d04c106f050088e8b036e7d8655b8254 /sys/crypto/cryptodev.c | |
| parent | strlcpy, strlcat (diff) | |
| download | wireguard-openbsd-121c0ac5c5f314dbfb18379b899a72365f7c7e15.tar.xz wireguard-openbsd-121c0ac5c5f314dbfb18379b899a72365f7c7e15.zip | |
You should always free the keys, not just on error --- the crypto
framework doesn't hold on to these.
Diffstat (limited to 'sys/crypto/cryptodev.c')
| -rw-r--r-- | sys/crypto/cryptodev.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/sys/crypto/cryptodev.c b/sys/crypto/cryptodev.c index 61ab161b9d0..d2f86852eaf 100644 --- a/sys/crypto/cryptodev.c +++ b/sys/crypto/cryptodev.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cryptodev.c,v 1.45 2002/06/07 08:09:04 nordin Exp $ */ +/* $OpenBSD: cryptodev.c,v 1.46 2002/06/10 21:09:27 angelos Exp $ */ /* * Copyright (c) 2001 Theo de Raadt @@ -276,12 +276,10 @@ cryptof_ioctl(fp, cmd, data, p) sop->ses = cse->ses; bail: - if (error) { - if (crie.cri_key) - FREE(crie.cri_key, M_XDATA); - if (cria.cri_key) - FREE(cria.cri_key, M_XDATA); - } + if (crie.cri_key) + FREE(crie.cri_key, M_XDATA); + if (cria.cri_key) + FREE(cria.cri_key, M_XDATA); break; case CIOCFSESSION: |