[RSLIB] BUG() when passing illegal parameters to decode_rs8() or decode_rs16()
Jörn Engel [Sat, 20 Oct 2007 21:14:42 +0000 (23:14 +0200)]
Returning -ERANGE should never happen.

Signed-off-by: Jörn Engel <joern@logfs.org>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>

lib/reed_solomon/decode_rs.c

index a58df56..65bc718 100644 (file)
@@ -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)