]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
3.2-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Feb 2012 21:26:55 +0000 (13:26 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Feb 2012 21:26:55 +0000 (13:26 -0800)
added patches:
arm-7326-2-pl330-fix-null-pointer-dereference-in-pl330_chan_ctrl.patch

queue-3.2/arm-7326-2-pl330-fix-null-pointer-dereference-in-pl330_chan_ctrl.patch [new file with mode: 0644]
queue-3.2/series

diff --git a/queue-3.2/arm-7326-2-pl330-fix-null-pointer-dereference-in-pl330_chan_ctrl.patch b/queue-3.2/arm-7326-2-pl330-fix-null-pointer-dereference-in-pl330_chan_ctrl.patch
new file mode 100644 (file)
index 0000000..34a2a7f
--- /dev/null
@@ -0,0 +1,42 @@
+From 46e33c606af8e0caeeca374103189663d877c0d6 Mon Sep 17 00:00:00 2001
+From: Javi Merino <javi.merino@arm.com>
+Date: Wed, 15 Feb 2012 17:36:39 +0100
+Subject: ARM: 7326/2: PL330: fix null pointer dereference in pl330_chan_ctrl()
+
+From: Javi Merino <javi.merino@arm.com>
+
+commit 46e33c606af8e0caeeca374103189663d877c0d6 upstream.
+
+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>
+
+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>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ arch/arm/common/pl330.c |    3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+--- a/arch/arm/common/pl330.c
++++ b/arch/arm/common/pl330.c
+@@ -1496,12 +1496,13 @@ int pl330_chan_ctrl(void *ch_id, enum pl
+       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);
index 82bf52504cdcffaa666d9f05494c09ee5ec76292..cc8368fd5f9aea689d078979cc732960067a3750 100644 (file)
@@ -8,3 +8,4 @@ regmap-fix-cache-defaults-initialization-from-raw-cache-defaults.patch
 ecryptfs-copy-up-lower-inode-attrs-after-setting-lower-xattr.patch
 correct-ktime-to-tod-clock-comparator-conversion.patch
 vfs-fix-d_inode_lookup-dentry-ref-leak.patch
+arm-7326-2-pl330-fix-null-pointer-dereference-in-pl330_chan_ctrl.patch