From: Dan Carpenter <[email protected]>

We dereference "ts" in the printk so move the kfree() down a line.

Signed-off-by: Dan Carpenter <[email protected]>
Acked-by: Alan Cox <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
---
 drivers/staging/cptm1217/clearpad_tm1217.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/cptm1217/clearpad_tm1217.c 
b/drivers/staging/cptm1217/clearpad_tm1217.c
index 269503f..76e4b78 100644
--- a/drivers/staging/cptm1217/clearpad_tm1217.c
+++ b/drivers/staging/cptm1217/clearpad_tm1217.c
@@ -460,9 +460,9 @@ static int cp_tm1217_probe(struct i2c_client *client,
        for (i = 0; i < TOUCH_SUPPORTED; i++) {
                input_dev = input_allocate_device();
                if (input_dev == NULL) {
-                       kfree(ts);
                        dev_err(ts->dev,
                                "cp_tm1217:Input Device Struct alloc failed\n");
+                       kfree(ts);
                        return -ENOMEM;
                }
                input_info = &ts->cp_input_info[i];
-- 
1.7.3.2

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to