mbox series

[v2,for-4.9.y,0/5] net/sched: init failure fixes

Message ID 1535007041-31605-1-git-send-email-amit.pundir@linaro.org
Headers show
Series net/sched: init failure fixes | expand

Message

Amit Pundir Aug. 23, 2018, 6:50 a.m. UTC
Hi Greg,

Kindly consider/review following net/sched fixes for stable 4.9.y.

This patchset is a follow-up of upstream fix
87b60cfacf9f ("net_sched: fix error recovery at qdisc creation")
cherry-picked on stable 4.9.y.

It fix null pointer dereferences due to uninitialized timer
(qdisc watchdog) or double frees due to ->destroy cleaning up a
second time. Here is the original submission
https://www.mail-archive.com/netdev@vger.kernel.org/msg186003.html

Cherry-picked and build tested on Linux 4.9.123 for ARCH=x86_64.

These fixes are applicable for stable 4.4.y kernel as well, but
one of the patches needed a minor rebasing, so I'm resending this
series for 4.4.y in a separate thread to avoid any confusion.

Regards,
Amit Pundir

Change since v1:
Rebased "sch_multiq: fix double free on init failure" patch
and fixed "unused variable" build warning.

Nikolay Aleksandrov (5):
  sch_htb: fix crash on init failure
  sch_multiq: fix double free on init failure
  sch_hhf: fix null pointer dereference on init failure
  sch_netem: avoid null pointer deref on init failure
  sch_tbf: fix two null pointer dereferences on init failure

 net/sched/sch_hhf.c    | 3 +++
 net/sched/sch_htb.c    | 5 +++--
 net/sched/sch_multiq.c | 9 ++-------
 net/sched/sch_netem.c  | 4 ++--
 net/sched/sch_tbf.c    | 5 +++--
 5 files changed, 13 insertions(+), 13 deletions(-)

-- 
2.7.4

Comments

Greg Kroah-Hartman Sept. 13, 2018, 12:30 p.m. UTC | #1
On Thu, Aug 23, 2018 at 12:20:36PM +0530, Amit Pundir wrote:
> Hi Greg,

> 

> Kindly consider/review following net/sched fixes for stable 4.9.y.

> 

> This patchset is a follow-up of upstream fix

> 87b60cfacf9f ("net_sched: fix error recovery at qdisc creation")

> cherry-picked on stable 4.9.y.

> 

> It fix null pointer dereferences due to uninitialized timer

> (qdisc watchdog) or double frees due to ->destroy cleaning up a

> second time. Here is the original submission

> https://www.mail-archive.com/netdev@vger.kernel.org/msg186003.html

> 

> Cherry-picked and build tested on Linux 4.9.123 for ARCH=x86_64.

> 

> These fixes are applicable for stable 4.4.y kernel as well, but

> one of the patches needed a minor rebasing, so I'm resending this

> series for 4.4.y in a separate thread to avoid any confusion.


Now applied for 4.4.y and 4.9.y, thanks.

greg k-h