Skip to contentSkip to conversation

Patch "irda: precedence bug in irlmp_seq_hb_idx()" has been added to the 4.2-stable tree

PATCH [email protected] 1 participant 1 message Nov 17, 2015

Subsystems (2)
lore
Share & tools
lore

[email protected]

author maintainer Applied
#0 lore raw

This is a note to let you know that I've just added the patch titled

irda: precedence bug in irlmp_seq_hb_idx()

to the 4.2-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
irda-precedence-bug-in-irlmp_seq_hb_idx.patch
and it can be found in the queue-4.2 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.

Quoted 1 line

From foo@baz Tue Nov 17 14:34:38 PST 2015

From: Dan Carpenter <[email protected]>
Date: Mon, 19 Oct 2015 13:16:49 +0300
Subject: irda: precedence bug in irlmp_seq_hb_idx()
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

From: Dan Carpenter <[email protected]>

[ Upstream commit 50010c20597d14667eff0fdb628309986f195230 ]

This is decrementing the pointer, instead of the value stored in the pointer. KASan detects it as an out of bounds reference.

---
 net/irda/irlmp.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
net/irda/irlmp.c +1 -1
@@ -1839,7 +1839,7 @@ static void *irlmp_seq_hb_idx(struct irl
 	for (element = hashbin_get_first(iter->hashbin);
 	     element != NULL;
 	     element = hashbin_get_next(iter->hashbin)) {
-		if (!off || *off-- == 0) {
+		if (!off || (*off)-- == 0) {
 			/* NB: hashbin left locked */
 			return element;
 		}

Patches currently in stable-queue which might be from [email protected] are

queue-4.2/irda-precedence-bug-in-irlmp_seq_hb_idx.patch