If the kmalloc() return NULL, the NULL pointer dereference will occur.
        new_ts->ts = ts;

Add exception check after the call to kmalloc() is made.

Signed-off-by: Austin Kim <austindh....@gmail.com>
---
 drivers/staging/media/meson/vdec/vdec_helpers.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/staging/media/meson/vdec/vdec_helpers.c 
b/drivers/staging/media/meson/vdec/vdec_helpers.c
index f16948b..e7e56d5 100644
--- a/drivers/staging/media/meson/vdec/vdec_helpers.c
+++ b/drivers/staging/media/meson/vdec/vdec_helpers.c
@@ -206,6 +206,10 @@ void amvdec_add_ts_reorder(struct amvdec_session *sess, 
u64 ts, u32 offset)
        unsigned long flags;
 
        new_ts = kmalloc(sizeof(*new_ts), GFP_KERNEL);
+       if (!new_ts) {
+               dev_err(sess->core->dev, "Failed to kmalloc()\n");
+               return;
+       }
        new_ts->ts = ts;
        new_ts->offset = offset;
 
-- 
2.6.2

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to