]> err.no Git - linux-2.6/commitdiff
V4L/DVB (4311): Fix possible dvb-pll oops
authorAndrew de Quincey <adq_dvb@lidskialf.net>
Sun, 16 Jul 2006 22:41:41 +0000 (19:41 -0300)
committerMauro Carvalho Chehab <mchehab@infradead.org>
Sat, 29 Jul 2006 20:22:25 +0000 (17:22 -0300)
Supplying a NULL i2c adapter to dvb_pll_attach is allowed, for example with
mt352 demods. However, the pll i2c probe will segfault because it does not
check for this.

Signed-off-by: Andrew de Quincey <adq_dvb@lidskialf.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
drivers/media/dvb/frontends/dvb-pll.c

index fce5c425225242cf3c16d232c5f39e4799a1e1fe..4c8a506479b4f1eb7dc11a8eb5be306b2f13559b 100644 (file)
@@ -619,14 +619,16 @@ int dvb_pll_attach(struct dvb_frontend *fe, int pll_addr, struct i2c_adapter *i2
        struct dvb_pll_priv *priv = NULL;
        int ret;
 
-       if (fe->ops.i2c_gate_ctrl)
-               fe->ops.i2c_gate_ctrl(fe, 1);
-
-       ret = i2c_transfer (i2c, msg, 2);
-       if (ret != 2)
-               return -1;
-       if (fe->ops.i2c_gate_ctrl)
-                    fe->ops.i2c_gate_ctrl(fe, 0);
+       if (i2c != NULL) {
+               if (fe->ops.i2c_gate_ctrl)
+                       fe->ops.i2c_gate_ctrl(fe, 1);
+
+               ret = i2c_transfer (i2c, msg, 2);
+               if (ret != 2)
+                       return -1;
+               if (fe->ops.i2c_gate_ctrl)
+                            fe->ops.i2c_gate_ctrl(fe, 0);
+       }
 
        priv = kzalloc(sizeof(struct dvb_pll_priv), GFP_KERNEL);
        if (priv == NULL)