Re: [PATCH RESEND net-next v4 6/9] net: ip: make ip_route_input_noref() return drop reasons
From: Paolo Abeni
Date: Tue Nov 05 2024 - 06:23:04 EST
Hi,
On 10/30/24 02:41, Menglong Dong wrote:
> @@ -175,10 +175,12 @@ static void ip_expire(struct timer_list *t)
>
> /* skb has no dst, perform route lookup again */
> iph = ip_hdr(head);
> - err = ip_route_input_noref(head, iph->daddr, iph->saddr, ip4h_dscp(iph),
> - head->dev);
> - if (err)
> + reason = ip_route_input_noref(head, iph->daddr, iph->saddr,
> + ip4h_dscp(iph), head->dev);
> + if (reason)
> goto out;
> + else
> + reason = SKB_DROP_REASON_FRAG_REASM_TIMEOUT;
I think the else branch above is confusing - and unneeded.
Please move the assignment after the comment below, so it's clear why we
get a TIMEOUT drop reason.
Thanks,
Paolo