lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 8 Jun 2016 14:31:33 +0200 From: Florian Westphal <fw@...len.de> To: <netdev@...r.kernel.org> Cc: Florian Westphal <fw@...len.de> Subject: [PATCH net] qfq: don't leak skb if kzalloc fails When we need to create a new aggregate to enqueue the skb we need to kzalloc the new struct. If that fails we returned ENOBUFS without freeing the skb. Spotted during code review. Signed-off-by: Florian Westphal <fw@...len.de> --- net/sched/sch_qfq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/sched/sch_qfq.c b/net/sched/sch_qfq.c index 8d2d8d9..435b970 100644 --- a/net/sched/sch_qfq.c +++ b/net/sched/sch_qfq.c @@ -1236,7 +1236,7 @@ static int qfq_enqueue(struct sk_buff *skb, struct Qdisc *sch) err = qfq_change_agg(sch, cl, cl->agg->class_weight, qdisc_pkt_len(skb)); if (err) - return err; + return qdisc_drop(skb, sch); } err = qdisc_enqueue(skb, cl->qdisc); -- 2.7.3
Powered by blists - more mailing lists