From 3c75efaf9b3cf25a70d4474778d3b8e4ceaf297d Mon Sep 17 00:00:00 2001 From: Abir Ghosh Date: Tue, 11 Apr 2017 10:10:23 +0530 Subject: [PATCH] qbt1000: Initialize drvdata structure before usage Fix uninitialized local variable error which might have lead to crash. CRs-Fixed: 2030137 Change-Id: I3fd95cb343c3175e4190c8ebfe209399db0602a6 Signed-off-by: Abir Ghosh Signed-off-by: Kota Priyanka --- drivers/soc/qcom/qbt1000.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/soc/qcom/qbt1000.c b/drivers/soc/qcom/qbt1000.c index b9e94b5d61dd..4e7b759ac810 100644 --- a/drivers/soc/qcom/qbt1000.c +++ b/drivers/soc/qcom/qbt1000.c @@ -318,6 +318,12 @@ static long qbt1000_ioctl( drvdata = file->private_data; + if (IS_ERR(priv_arg)) { + dev_err(drvdata->dev, "%s: invalid user space pointer %lu\n", + __func__, arg); + return -EINVAL; + } + mutex_lock(&drvdata->mutex); pr_debug("%s %d\n", __func__, cmd);