diff options
| author | Kumar Sanghvi <kumar.sanghvi@stericsson.com> | 2011-04-06 11:58:13 +0530 |
|---|---|---|
| committer | Philippe Langlais <philippe.langlais@stericsson.com> | 2011-12-06 11:00:57 +0100 |
| commit | 74779bb344ae1f3063a04118569e4ac428f42443 (patch) | |
| tree | 16fdf87dcf397e7d8d2535124623ec76897489b4 | |
| parent | 2c0cda17d23089167e6a120c24fe0167f2cbd793 (diff) | |
u8500:shrm: Fix null pointer dereference
Fixes null pointer dereference in shrm char interface
ST-Ericsson Linux next: -
ST-Ericsson ID: ER332892
ST-Ericsson FOSS-OUT ID: Trivial
Change-Id: Ia60fa08e61e50c2e2645e22c2d7b9c5df01df7ef
Signed-off-by: Kumar Sanghvi <kumar.sanghvi@stericsson.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/20103
Reviewed-by: Hemant-vilas RAMDASI <hemant.ramdasi@stericsson.com>
Reviewed-by: Arun MURTHY <arun.murthy@stericsson.com>
Reviewed-by: Jonas ABERG <jonas.aberg@stericsson.com>
| -rw-r--r-- | drivers/char/shrm_char.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/char/shrm_char.c b/drivers/char/shrm_char.c index 714d38e42f1..e2aaf60674f 100644 --- a/drivers/char/shrm_char.c +++ b/drivers/char/shrm_char.c @@ -273,6 +273,7 @@ int get_size_of_new_msg(struct message_queue *q) struct queue_element *new_msg = NULL; struct list_head *msg_list; struct shrm_dev *shrm = q->shrm; + int size = 0; dev_dbg(shrm->dev, "%s IN\n", __func__); @@ -284,12 +285,13 @@ int get_size_of_new_msg(struct message_queue *q) dev_err(shrm->dev, "no message found\n"); return -EFAULT; } + size = new_msg->size; break; } spin_unlock_bh(&q->update_lock); dev_dbg(shrm->dev, "%s OUT\n", __func__); - return new_msg->size; + return size; } /** |