amt: fix return value of amt_update_handler()
authorTaehee Yoo <ap420073@gmail.com>
Mon, 23 May 2022 16:17:07 +0000 (16:17 +0000)
committerJakub Kicinski <kuba@kernel.org>
Thu, 26 May 2022 04:36:16 +0000 (21:36 -0700)
If a relay receives an update message, it lookup a tunnel.
and if there is no tunnel for that message, it should be treated
as an error, not a success.
But amt_update_handler() returns false, which means success.

Fixes: cbc21dc1cfe9 ("amt: add data plane of amt interface")
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/amt.c

index f41668d..635de07 100644 (file)
@@ -2423,7 +2423,7 @@ static bool amt_update_handler(struct amt_dev *amt, struct sk_buff *skb)
                }
        }
 
-       return false;
+       return true;
 
 report:
        iph = ip_hdr(skb);