diff options
author | Jörn Engel <joern@logfs.org> | 2007-10-20 23:14:42 +0200 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2007-10-20 22:29:09 +0100 |
commit | 1dd7fdb163645f453f5ae55686511b6fcc2314cd (patch) | |
tree | bf07861981b8dac7a01286e9ef21d96618254ee6 /lib | |
parent | c32b8dcc45c823cf30dcdf0fc37ee2f6b78f38cb (diff) | |
download | op-kernel-dev-1dd7fdb163645f453f5ae55686511b6fcc2314cd.zip op-kernel-dev-1dd7fdb163645f453f5ae55686511b6fcc2314cd.tar.gz |
[RSLIB] BUG() when passing illegal parameters to decode_rs8() or decode_rs16()
Returning -ERANGE should never happen.
Signed-off-by: Jörn Engel <joern@logfs.org>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/reed_solomon/decode_rs.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/reed_solomon/decode_rs.c b/lib/reed_solomon/decode_rs.c index a58df56..65bc718 100644 --- a/lib/reed_solomon/decode_rs.c +++ b/lib/reed_solomon/decode_rs.c @@ -39,8 +39,7 @@ /* Check length parameter for validity */ pad = nn - nroots - len; - if (pad < 0 || pad >= nn) - return -ERANGE; + BUG_ON(pad < 0 || pad >= nn); /* Does the caller provide the syndrome ? */ if (s != NULL) |