[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20160903170417.14061-1-colin.king@canonical.com>
Date: Sat, 3 Sep 2016 18:04:17 +0100
From: Colin King <colin.king@...onical.com>
To: Jemma Denson <jdenson@...il.com>,
Patrick Boettcher <patrick.boettcher@...teo.de>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
linux-media@...r.kernel.org
Cc: linux-kernel@...r.kernel.org
Subject: [PATCH] [media] cx24120: do not allow an invalid delivery system types
From: Colin Ian King <colin.king@...onical.com>
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 <colin.king@...onical.com>
---
drivers/media/dvb-frontends/cx24120.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/media/dvb-frontends/cx24120.c b/drivers/media/dvb-frontends/cx24120.c
index 066ee38..3112a32 100644
--- a/drivers/media/dvb-frontends/cx24120.c
+++ b/drivers/media/dvb-frontends/cx24120.c
@@ -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;
--
2.9.3
Powered by blists - more mailing lists