Hi Oleksij,
the topic said what this patch does - but there's no commit message that
tells the reader why the check can be dropped.
And the removed comment in the patch leaves me even more lost ...
Regards,
Oliver
On 09/07/2019 10.58, Oleksij Rempel wrote:
Signed-off-by: Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx>
---
net/can/j1939/socket.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/net/can/j1939/socket.c b/net/can/j1939/socket.c
index 19e662574b5f..39df867cfe84 100644
--- a/net/can/j1939/socket.c
+++ b/net/can/j1939/socket.c
@@ -272,14 +272,9 @@ static bool j1939_sk_recv_match_one(struct j1939_sock *jsk,
static void j1939_sk_recv_one(struct j1939_sock *jsk, struct sk_buff *oskb)
{
const struct j1939_sk_buff_cb *oskcb = j1939_skb_to_cb(oskb);
- const struct can_skb_priv *oskb_prv = can_skb_prv(oskb);
struct j1939_sk_buff_cb *skcb;
struct sk_buff *skb;
- if (jsk->ifindex != oskb_prv->ifindex)
- /* this socket does not take packets from this iface */
- return;
-
if (!j1939_sk_recv_match_one(jsk, oskcb))
return;
