[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

[PATCH] sch_sfb: Fix missing NULL check

From: Alan Cox <hidden>
Date: 2012-07-12 13:22:24

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);

Re: [PATCH] sch_sfb: Fix missing NULL check

From: David Miller <davem@davemloft.net>
Date: 2012-07-12 13:25:57

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.

Re: [PATCH] sch_sfb: Fix missing NULL check

From: Eric Dumazet <hidden>
Date: 2012-07-12 13:50:26

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>

Re: [PATCH] sch_sfb: Fix missing NULL check

From: David Miller <davem@davemloft.net>
Date: 2012-07-12 15:33:57

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 :-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help