2017-11-28 18:18:49

by Tommi Rantala

[permalink] [raw]
Subject: Re: [PATCH] tipc: call tipc_rcv() only if bearer is up in tipc_udp_recv()

On 28.11.2017 16:58, David Miller wrote:
> From: Tommi Rantala <[email protected]>
> Date: Tue, 28 Nov 2017 14:53:15 +0200
>
>> -
>> - if (unlikely(msg_user(hdr) == LINK_CONFIG)) {
>> - err = tipc_udp_rcast_disc(b, skb);
>> - if (err)
>> - goto rcu_out;
>> + } else {
>> + if (unlikely(b && msg_user(hdr) == LINK_CONFIG))
>> + tipc_udp_rcast_disc(b, skb);
>> + kfree_skb(skb);
>> }
>
> Either put the 'err' propagation back or clearly explain in your
> commit log message why this part of the change if absolutely essential
> for this bug fix.
>
> Thank you.
>

Thanks for the feedback. I'll post patch v2 soon.

-Tommi

From 1585325073293590077@xxx Tue Nov 28 15:43:25 +0000 2017
X-GM-THRID: 1585314493852004005
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread