summaryrefslogtreecommitdiffstats
path: root/arch/arm
diff options
context:
space:
mode:
authorJavi Merino <javi.merino@arm.com>2012-02-15 17:36:39 +0100
committerRussell King <rmk+kernel@arm.linux.org.uk>2012-02-15 21:10:49 +0000
commit46e33c606af8e0caeeca374103189663d877c0d6 (patch)
treea1f97d7fd9af85b6f505c166222152192c11c1bb /arch/arm
parent4272f98a1ae81709fc5c804c33c044064e419cd9 (diff)
downloadop-kernel-dev-46e33c606af8e0caeeca374103189663d877c0d6.zip
op-kernel-dev-46e33c606af8e0caeeca374103189663d877c0d6.tar.gz
ARM: 7326/2: PL330: fix null pointer dereference in pl330_chan_ctrl()
This fixes the thrd->req_running field being accessed before thrd is checked for null. The error was introduced in abb959f: ARM: 7237/1: PL330: Fix driver freeze Reference: <1326458191-23492-1-git-send-email-mans.rullgard@linaro.org> Cc: stable@kernel.org Signed-off-by: Mans Rullgard <mans.rullgard@linaro.org> Acked-by: Javi Merino <javi.merino@arm.com> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm')
-rw-r--r--arch/arm/common/pl330.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/arm/common/pl330.c b/arch/arm/common/pl330.c
index d8e44a4..ff3ad22 100644
--- a/arch/arm/common/pl330.c
+++ b/arch/arm/common/pl330.c
@@ -1502,12 +1502,13 @@ int pl330_chan_ctrl(void *ch_id, enum pl330_chan_op op)
struct pl330_thread *thrd = ch_id;
struct pl330_dmac *pl330;
unsigned long flags;
- int ret = 0, active = thrd->req_running;
+ int ret = 0, active;
if (!thrd || thrd->free || thrd->dmac->state == DYING)
return -EINVAL;
pl330 = thrd->dmac;
+ active = thrd->req_running;
spin_lock_irqsave(&pl330->lock, flags);
OpenPOWER on IntegriCloud