[PATCH] sch_sfb: Fix missing NULL check
Subsystems:
networking [general], tc subsystem, the rest
STALE5144d
4 messages,
3 authors,
2012-07-12 · open the first message on its own page
From: Alan Cox <redacted>
Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=44461
Signed-off-by: Alan Cox <redacted>
---
net/sched/sch_sfb.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/net/sched/sch_sfb.c b/net/sched/sch_sfb.c
index 74305c8..30ea467 100644
--- a/net/sched/sch_sfb.c
+++ b/net/sched/sch_sfb.c
@@ -570,6 +570,8 @@ static int sfb_dump(struct Qdisc *sch, struct sk_buff *skb)
sch->qstats.backlog = q->qdisc->qstats.backlog;
opts = nla_nest_start(skb, TCA_OPTIONS);
+ if (opts == NULL)
+ goto nla_put_failure;
if (nla_put(skb, TCA_SFB_PARMS, sizeof(opt), &opt))
goto nla_put_failure;
return nla_nest_end(skb, opts);
From: Alan Cox <redacted>
Date: Thu, 12 Jul 2012 14:39:11 +0100
Signed-off-by: Alan Cox <redacted>
^^^^
I'm truly astonished that you type in signoffs by hand Alan.
On Thu, 2012-07-12 at 06:25 -0700, David Miller wrote:
From: Alan Cox <redacted>
Date: Thu, 12 Jul 2012 14:39:11 +0100
quoted
Signed-off-by: Alan Cox <redacted>
^^^^
I'm truly astonished that you type in signoffs by hand Alan.
Weel, I do the same ;)
Feel free to add my
Acked-by: Eric Dumazet <edumazet@google.com>
From: Eric Dumazet <redacted>
Date: Thu, 12 Jul 2012 15:50:21 +0200
On Thu, 2012-07-12 at 06:25 -0700, David Miller wrote:
quoted
From: Alan Cox <redacted>
Date: Thu, 12 Jul 2012 14:39:11 +0100
quoted
Signed-off-by: Alan Cox <redacted>
^^^^
I'm truly astonished that you type in signoffs by hand Alan.
Weel, I do the same ;)
You guys are weird :-)
Feel free to add my
Acked-by: Eric Dumazet <edumazet@google.com>
Applied, with signoff typo fixed too :-)