Patch "cxl: fix potential NULL dereference in free_adapter()" has been added to the 4.7-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
cxl: fix potential NULL dereference in free_adapter()
to the 4.7-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:
cxl-fix-potential-null-dereference-in-free_adapter.patch
and it can be found in the queue-4.7 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
8fbaa51d43ef2c6a72849ec34060910723a0365fMon Sep 17 00:00:00 2001
From: Andrew Donnellan <[email protected]>
Date: Fri, 15 Jul 2016 17:20:36 +1000
Subject: cxl: fix potential NULL dereference in free_adapter()
From: Andrew Donnellan <[email protected]>
commit 8fbaa51d43ef2c6a72849ec34060910723a0365f upstream.
If kzalloc() fails when allocating adapter->guest in cxl_guest_init_adapter(), we call free_adapter() before erroring out. free_adapter() in turn attempts to dereference adapter->guest, which in this case is NULL.
In free_adapter(), skip the adapter->guest cleanup if adapter->guest is NULL.
--- drivers/misc/cxl/guest.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-)
drivers/misc/cxl/guest.c +9 -7
@@ -1052,16 +1052,18 @@ static void free_adapter(struct cxl *adastruct irq_avail *cur;int i; -if (adapter->guest->irq_avail) { -for (i =0 ; i < adapter->guest->irq_nranges; i++) { - cur = &adapter->guest->irq_avail[i]; -kfree (cur->bitmap); +if (adapter->guest) { +if (adapter->guest->irq_avail) { +for (i =0 ; i < adapter->guest->irq_nranges; i++) { + cur = &adapter->guest->irq_avail[i]; +kfree (cur->bitmap); + } +kfree (adapter->guest->irq_avail); } -kfree (adapter->guest->irq_avail); +kfree (adapter->guest->status); +kfree (adapter->guest); } -kfree (adapter->guest->status);cxl_remove_adapter_nr (adapter); -kfree (adapter->guest);kfree (adapter); }
Patches currently in stable-queue which might be from [email protected] are
queue-4.7/cxl-fix-potential-null-dereference-in-free_adapter.patch