gpu: check the null pointer [1/1]
authorLiang Ji <liang.ji@amlogic.com>
Tue, 2 Jun 2020 07:18:04 +0000 (15:18 +0800)
committerSky Zhou <sky.zhou@amlogic.com>
Thu, 27 Aug 2020 11:57:35 +0000 (04:57 -0700)
PD#SWPL-31635

Problem:
mali crash because of null pointer

Solution:
add ctx check

Verify:
AH212

Change-Id: Ic712e9bc99d81a65b6c0dbad0ebcfb9378cc2b46
Signed-off-by: Dezhi Kong <dezhi.kong@amlogic.com>
dvalin/kernel/drivers/gpu/arm/midgard/mali_kbase_mem_profile_debugfs.c

index 5d38ed2412111ab2fcbd5b10cc14e996b30e8868..5752d4a0fe8ca135cc88faf8eaaba155963ab6c3 100644 (file)
@@ -38,6 +38,9 @@ static int kbasep_mem_profile_seq_show(struct seq_file *sfile, void *data)
 {
        struct kbase_context *kctx = sfile->private;
 
+       if (IS_ERR_OR_NULL(kctx->kctx_dentry)) /* not initialized */
+               return -EINVAL;
+
        mutex_lock(&kctx->mem_profile_lock);
 
        seq_write(sfile, kctx->mem_profile_data, kctx->mem_profile_size);
@@ -70,6 +73,9 @@ int kbasep_mem_profile_debugfs_insert(struct kbase_context *kctx, char *data,
 {
        int err = 0;
 
+       if (IS_ERR_OR_NULL(kctx->kctx_dentry)) /* not initialized */
+               return -EINVAL;
+
        mutex_lock(&kctx->mem_profile_lock);
 
        dev_dbg(kctx->kbdev->dev, "initialised: %d",
@@ -106,6 +112,9 @@ int kbasep_mem_profile_debugfs_insert(struct kbase_context *kctx, char *data,
 
 void kbasep_mem_profile_debugfs_remove(struct kbase_context *kctx)
 {
+       if (IS_ERR_OR_NULL(kctx->kctx_dentry)) /* not initialized */
+               return ;
+
        mutex_lock(&kctx->mem_profile_lock);
 
        dev_dbg(kctx->kbdev->dev, "initialised: %d",