[media] cx24120: do not allow an invalid delivery system types
authorColin Ian King <[email protected]>
Sat, 3 Sep 2016 17:04:17 +0000 (14:04 -0300)
committerMauro Carvalho Chehab <[email protected]>
Mon, 24 Oct 2016 19:00:42 +0000 (17:00 -0200)
cx24120_set_frontend currently allows invalid delivery system types
other than SYS_DVBS2 and SYS_DVBS.  Fix this by returning -EINVAL
for invalid values.

Signed-off-by: Colin Ian King <[email protected]>
Acked-by: Jemma Denson <[email protected]>
drivers/media/dvb-frontends/cx24120.c

index 066ee387bf250894b1067b023b2d1d760790ad02..3112a3206d4670d518be371611d07d82e21fc030 100644 (file)
@@ -1154,8 +1154,7 @@ static int cx24120_set_frontend(struct dvb_frontend *fe)
                dev_dbg(&state->i2c->dev,
                        "delivery system(%d) not supported\n",
                        c->delivery_system);
-               ret = -EINVAL;
-               break;
+               return -EINVAL;
        }
 
        state->dnxt.delsys = c->delivery_system;