diff options
| author | riastradh <riastradh@NetBSD.org> | 2022-05-21 23:11:03 +0000 |
|---|---|---|
| committer | riastradh <riastradh@NetBSD.org> | 2022-05-21 23:11:03 +0000 |
| commit | 22ec612debdca25309d8354ea8f3c1cb129dfdbb (patch) | |
| tree | c5244f391d24442c60b6c3431f01f05ff074f7a1 | |
| parent | 7bfc0d99876f47ee36bea0a028dd1554cc15adf6 (diff) | |
crypto(4): Fix set-but-unused variable warning.
This deliberately ignores the error code returned by crypto_dispatch,
but that error code is fundamentally incoherent and the issue will be
mooted by subsequent changes to make it return void and always pass
the error through the callback, as well as subsequent changes to rip
out the EAGAIN logic anyway.
| -rw-r--r-- | sys/opencrypto/cryptodev.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/sys/opencrypto/cryptodev.c b/sys/opencrypto/cryptodev.c index b05cba4fea5..e87bd96173d 100644 --- a/sys/opencrypto/cryptodev.c +++ b/sys/opencrypto/cryptodev.c @@ -1,4 +1,4 @@ -/* $NetBSD: cryptodev.c,v 1.114 2022/05/21 20:37:18 riastradh Exp $ */ +/* $NetBSD: cryptodev.c,v 1.115 2022/05/21 23:11:03 riastradh Exp $ */ /* $FreeBSD: src/sys/opencrypto/cryptodev.c,v 1.4.2.4 2003/06/03 00:09:02 sam Exp $ */ /* $OpenBSD: cryptodev.c,v 1.53 2002/07/10 22:21:30 mickey Exp $ */ @@ -64,7 +64,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: cryptodev.c,v 1.114 2022/05/21 20:37:18 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: cryptodev.c,v 1.115 2022/05/21 23:11:03 riastradh Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -737,13 +737,12 @@ static int cryptodev_mcb(struct cryptop *crp) { struct csession *cse = crp->crp_opaque; - int error = 0; mutex_enter(&cryptodev_mtx); cse->error = crp->crp_etype; if (crp->crp_etype == EAGAIN) { mutex_exit(&cryptodev_mtx); - error = crypto_dispatch(crp); + (void)crypto_dispatch(crp); mutex_enter(&cryptodev_mtx); } |
