aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorSuman Anna <s-anna@ti.com>2012-05-16 16:09:18 -0500
committerZiyann <jaraidaniel@gmail.com>2014-10-01 12:58:34 +0200
commite623b294933a8a8c6606a24b7ca343256ddc0f92 (patch)
tree7767e0e39bb7bd362163ef682f1b3b0f25dd7029 /drivers
parent9b2d0601435c14add3ee1be73a7d9e3451cee583 (diff)
downloadkernel_samsung_tuna-e623b294933a8a8c6606a24b7ca343256ddc0f92.zip
kernel_samsung_tuna-e623b294933a8a8c6606a24b7ca343256ddc0f92.tar.gz
kernel_samsung_tuna-e623b294933a8a8c6606a24b7ca343256ddc0f92.tar.bz2
gpu: ion: check for input arguments in ion_free
ion_free is a public API and is directly dereferencing a pointer without checking for NULL. A check has been added for the same and throws a WARN in case of invalid arguments. Change-Id: Idcf72e4449f8cba2b8342d6fb0f64600c2c59c7e Signed-off-by: Suman Anna <s-anna@ti.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/gpu/ion/ion.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/ion/ion.c b/drivers/gpu/ion/ion.c
index eb072fc..2bb559b3 100644
--- a/drivers/gpu/ion/ion.c
+++ b/drivers/gpu/ion/ion.c
@@ -366,6 +366,8 @@ void ion_free(struct ion_client *client, struct ion_handle *handle)
{
bool valid_handle;
+ if (WARN_ON(!client || !handle))
+ return;
BUG_ON(client != handle->client);
mutex_lock(&client->lock);