* [PATCH] crypto: omap-des: fix BUG_ON condition
@ 2015-01-05 8:23 Asaf Vertz
2015-01-08 10:48 ` Herbert Xu
0 siblings, 1 reply; 2+ messages in thread
From: Asaf Vertz @ 2015-01-05 8:23 UTC (permalink / raw)
To: herbert, davem; +Cc: linux-crypto, linux-kernel
dd->total is unsigned so it won't do any good to check for negative value after subtracting
instead of that we should check if the subtracted value is bigger than him
This was partially found by using a static code analysis program called cppcheck.
Signed-off-by: Asaf Vertz <asaf.vertz@tandemg.com>
---
drivers/crypto/omap-des.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/crypto/omap-des.c b/drivers/crypto/omap-des.c
index e350f5b..0b8dcf5 100644
--- a/drivers/crypto/omap-des.c
+++ b/drivers/crypto/omap-des.c
@@ -965,9 +965,9 @@ static irqreturn_t omap_des_irq(int irq, void *dev_id)
}
}
- dd->total -= DES_BLOCK_SIZE;
+ BUG_ON(dd->total < DES_BLOCK_SIZE);
- BUG_ON(dd->total < 0);
+ dd->total -= DES_BLOCK_SIZE;
/* Clear IRQ status */
status &= ~DES_REG_IRQ_DATA_OUT;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] crypto: omap-des: fix BUG_ON condition
2015-01-05 8:23 [PATCH] crypto: omap-des: fix BUG_ON condition Asaf Vertz
@ 2015-01-08 10:48 ` Herbert Xu
0 siblings, 0 replies; 2+ messages in thread
From: Herbert Xu @ 2015-01-08 10:48 UTC (permalink / raw)
To: Asaf Vertz; +Cc: davem, linux-crypto, linux-kernel
On Mon, Jan 05, 2015 at 10:23:10AM +0200, Asaf Vertz wrote:
> dd->total is unsigned so it won't do any good to check for negative value after subtracting
> instead of that we should check if the subtracted value is bigger than him
>
> This was partially found by using a static code analysis program called cppcheck.
>
> Signed-off-by: Asaf Vertz <asaf.vertz@tandemg.com>
Applied.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-01-08 10:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-05 8:23 [PATCH] crypto: omap-des: fix BUG_ON condition Asaf Vertz
2015-01-08 10:48 ` Herbert Xu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).