summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndi Shyti <andi@etezian.org>2014-01-30 04:06:41 +0100
committerMauro Carvalho Chehab <m.chehab@samsung.com>2014-02-04 09:36:03 +0100
commita33dd5171d141c378df498aba3fa3c9a573cacb6 (patch)
tree5a53601adccf1c91734d1619fcf58887f7da94fa
parent[media] cx24117: remove dead code in always 'false' if statement (diff)
downloadlinux-a33dd5171d141c378df498aba3fa3c9a573cacb6.tar.xz
linux-a33dd5171d141c378df498aba3fa3c9a573cacb6.zip
[media] cx24117: use a valid dev pointer for dev_err printout
Don't use '&state->priv->i2c->dev' reference to device because state is still 'NULL'. Use '&i2c->dev' instead. This bug has been reported by scan.coverity.com Signed-off-by: Andi Shyti <andi@etezian.org> Signed-off-by: Michael Krufky <mkrufky@linuxtv.org> Cc: vger@stable.kernel.org Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
-rw-r--r--drivers/media/dvb-frontends/cx24117.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb-frontends/cx24117.c b/drivers/media/dvb-frontends/cx24117.c
index 782dcfe6ae69..a6c3c9e2e897 100644
--- a/drivers/media/dvb-frontends/cx24117.c
+++ b/drivers/media/dvb-frontends/cx24117.c
@@ -1176,7 +1176,7 @@ struct dvb_frontend *cx24117_attach(const struct cx24117_config *config,
switch (demod) {
case 0:
- dev_err(&state->priv->i2c->dev,
+ dev_err(&i2c->dev,
"%s: Error attaching frontend %d\n",
KBUILD_MODNAME, demod);
goto error1;