Input: cyttsp - remove useless checks in cyttsp_probe()
authorDmitry Torokhov <dmitry.torokhov@gmail.com>
Sun, 4 Mar 2012 17:26:15 +0000 (09:26 -0800)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Sun, 4 Mar 2012 18:49:27 +0000 (10:49 -0800)
This fixes reference-before-check problem; there is no reason to check
if caller passed NULL dev or bus_ops as it is done only by bus-specific
drivers which already do the right thing.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
drivers/input/touchscreen/cyttsp_core.c

index 8be22479b41c60642d5a703de709b035fdb12bb4..f030d9ec795d0ae062a76260ff2943a28101e384 100644 (file)
@@ -523,7 +523,7 @@ struct cyttsp *cyttsp_probe(const struct cyttsp_bus_ops *bus_ops,
        struct input_dev *input_dev;
        int error;
 
-       if (!dev || !bus_ops || !pdata || !pdata->name || irq <= 0) {
+       if (!pdata || !pdata->name || irq <= 0) {
                error = -EINVAL;
                goto err_out;
        }