]> www.infradead.org Git - nvme.git/commit
ipv4: Fix incorrect source address in Record Route option
authorIdo Schimmel <idosch@nvidia.com>
Thu, 18 Jul 2024 12:34:07 +0000 (15:34 +0300)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 23 Jul 2024 08:28:16 +0000 (10:28 +0200)
commitcc73bbab4b1fb8a4f53a24645871dafa5f81266a
tree0ab73811644dbe8ec42c2cba292e5b8d2679b2bd
parentd7e78951a8b8b53e4d52c689d927a6887e6cfadf
ipv4: Fix incorrect source address in Record Route option

The Record Route IP option records the addresses of the routers that
routed the packet. In the case of forwarded packets, the kernel performs
a route lookup via fib_lookup() and fills in the preferred source
address of the matched route.

The lookup is performed with the DS field of the forwarded packet, but
using the RT_TOS() macro which only masks one of the two ECN bits. If
the packet is ECT(0) or CE, the matched route might be different than
the route via which the packet was forwarded as the input path masks
both of the ECN bits, resulting in the wrong address being filled in the
Record Route option.

Fix by masking both of the ECN bits.

Fixes: 8e36360ae876 ("ipv4: Remove route key identity dependencies in ip_rt_get_source().")
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Reviewed-by: Guillaume Nault <gnault@redhat.com>
Link: https://patch.msgid.link/20240718123407.434778-1-idosch@nvidia.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
net/ipv4/route.c