Re: Re: [net PATCH] net: sched: Fix one possible panic when no destroy callback

On Tue, Jun 27, 2017 at 5:54 PM, Gao Feng [off-list ref] wrote:

At 2017-06-28 01:49:50, "Eric Dumazet" [off-list ref] wrote:

quoted

On Tue, 2017-06-27 at 10:08 -0700, Cong Wang wrote:

quoted

On Tue, Jun 27, 2017 at 9:50 AM, Eric Dumazet [off-list ref] wrote:

quoted

On Tue, 2017-06-27 at 09:30 -0700, Cong Wang wrote:

quoted

On Mon, Jun 26, 2017 at 6:35 PM,  [off-list ref] wrote:

quoted

From: Gao Feng <redacted>

When qdisc fail to init, qdisc_create would invoke the destroy callback
to cleanup. But there is no check if the callback exists really. So it
would cause the panic if there is no real destroy callback like these
qdisc codel, pfifo, pfifo_fast, and so on.

Now add one the check for destroy to avoid the possible panic.

Signed-off-by: Gao Feng <redacted>

Looks good,

Acked-by: Cong Wang <redacted>

This is introduced by commit 87b60cfacf9f17cf71933c6e33b.
Please add proper Fixes tag next time.

OK. Actually I didn't know it is introduced by this commit before :)
Need I send an update patch again ?

Yes please, update the changelog as Eric suggested and also
add Fixes.