Re: [PATCH 4.8 78/92] batman-adv: fix splat on disabling an interface

[Sven Eckelmann]

On Donnerstag, 17. November 2016 11:32:51 CET Greg Kroah-Hartman wrote:
> 4.8-stable review patch.  If anyone has any objections, please let me know.
> 
> ------------------
> 
> From: Linus Lüssing <linus.luessing@xxxxxxxxx>
> 
> commit 9799c50372b23ed774791bdb87d700f1286ee8a9 upstream.
> 
> As long as there is still a reference for a hard interface held, there might
> still be a forwarding packet relying on its attributes.
> 
> Therefore avoid setting hard_iface->soft_iface to NULL when disabling a hard
> interface.
> 
> This fixes the following, potential splat:
> 
[...]

Thanks for your backporting efforts. There is already a revert for this patch
queued up in batadv/net [1] (targets net.git). So maybe you want to drop this
patch from your stable queue.

We also got test results from Linus which showed that this problem didn't
happen anymore with recent kernel versions (without his patch). These results
made him also Ack the revert.

Sorry about this confusing situation.

Kind regards,
	Sven

[1] https://git.open-mesh.org/linux-merge.git/commit/27915aa61060fd8954a68a86657784705955088a

Attachment: signature.asc
Description: This is a digitally signed message part.