Jianguo Wu
d62df86c17
seg6: fix parameter passing when calling NF_HOOK() in End.DX4 and End.DX6 behaviors
...
[ Upstream commit 9a3bc8d16e0aacd65c31aaf23a2bced3288a7779 ]
input_action_end_dx4() and input_action_end_dx6() are called NF_HOOK() for
PREROUTING hook, in PREROUTING hook, we should passing a valid indev,
and a NULL outdev to NF_HOOK(), otherwise may trigger a NULL pointer
dereference, as below:
[74830.647293] BUG: kernel NULL pointer dereference, address: 0000000000000090
[74830.655633] #PF: supervisor read access in kernel mode
[74830.657888] #PF: error_code(0x0000) - not-present page
[74830.659500] PGD 0 P4D 0
[74830.660450] Oops: 0000 [#1 ] PREEMPT SMP PTI
...
[74830.664953] Hardware name: Red Hat KVM, BIOS 0.5.1 01/01/2011
[74830.666569] RIP: 0010:rpfilter_mt+0x44/0x15e [ipt_rpfilter]
...
[74830.689725] Call Trace:
[74830.690402] <IRQ>
[74830.690953] ? show_trace_log_lvl+0x1c4/0x2df
[74830.692020] ? show_trace_log_lvl+0x1c4/0x2df
[74830.693095] ? ipt_do_table+0x286/0x710 [ip_tables]
[74830.694275] ? __die_body.cold+0x8/0xd
[74830.695205] ? page_fault_oops+0xac/0x140
[74830.696244] ? exc_page_fault+0x62/0x150
[74830.697225] ? asm_exc_page_fault+0x22/0x30
[74830.698344] ? rpfilter_mt+0x44/0x15e [ipt_rpfilter]
[74830.699540] ipt_do_table+0x286/0x710 [ip_tables]
[74830.700758] ? ip6_route_input+0x19d/0x240
[74830.701752] nf_hook_slow+0x3f/0xb0
[74830.702678] input_action_end_dx4+0x19b/0x1e0
[74830.703735] ? input_action_end_t+0xe0/0xe0
[74830.704734] seg6_local_input_core+0x2d/0x60
[74830.705782] lwtunnel_input+0x5b/0xb0
[74830.706690] __netif_receive_skb_one_core+0x63/0xa0
[74830.707825] process_backlog+0x99/0x140
[74830.709538] __napi_poll+0x2c/0x160
[74830.710673] net_rx_action+0x296/0x350
[74830.711860] __do_softirq+0xcb/0x2ac
[74830.713049] do_softirq+0x63/0x90
input_action_end_dx4() passing a NULL indev to NF_HOOK(), and finally
trigger a NULL dereference in rpfilter_mt()->rpfilter_is_loopback():
static bool
rpfilter_is_loopback(const struct sk_buff *skb,
const struct net_device *in)
{
// in is NULL
return skb->pkt_type == PACKET_LOOPBACK ||
in->flags & IFF_LOOPBACK;
}
Fixes: 7a3f5b0de364 ("netfilter: add netfilter hooks to SRv6 data plane")
Signed-off-by: Jianguo Wu <wujianguo@chinatelecom.cn>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
2024-06-27 13:49:08 +02:00
..
2023-08-02 12:28:16 -07:00
2024-04-17 11:19:30 +02:00
2024-02-05 20:14:36 +00:00
2024-05-17 12:02:24 +02:00
2022-11-07 12:26:15 +00:00
2024-03-01 13:35:04 +01:00
2023-02-13 18:35:15 +08:00
2023-07-28 11:01:56 +01:00
2023-08-16 11:09:16 +01:00
2023-06-22 18:40:38 -07:00
2024-04-03 15:28:35 +02:00
2023-05-24 08:43:39 +01:00
2024-03-01 13:35:10 +01:00
2024-05-17 12:02:23 +02:00
2023-08-29 17:39:15 -07:00
2023-03-17 08:25:05 +00:00
2023-07-31 13:53:10 -07:00
2024-06-21 14:38:15 +02:00
2022-08-29 12:47:15 +01:00
2024-06-21 14:38:20 +02:00
2023-03-21 21:32:18 -07:00
2024-04-13 13:07:30 +02:00
2024-05-17 12:02:24 +02:00
2024-05-17 12:02:07 +02:00
2024-05-17 12:02:24 +02:00
2024-04-13 13:07:30 +02:00
2024-04-13 13:07:30 +02:00
2023-08-30 09:44:09 +01:00
2022-09-29 07:18:00 +02:00
2023-08-20 15:17:47 +01:00
2023-08-16 10:03:08 +01:00
2024-04-03 15:28:39 +02:00
2022-09-29 07:18:01 +02:00
2024-04-03 15:28:39 +02:00
2022-05-16 13:03:29 +02:00
2022-11-18 02:15:22 +01:00
2024-01-31 16:19:04 -08:00
2024-04-03 15:28:39 +02:00
2024-04-03 15:28:39 +02:00
2024-06-12 11:11:51 +02:00
2024-06-27 13:49:06 +02:00
2023-07-12 17:12:29 -07:00
2023-06-19 11:32:58 -07:00
2024-06-12 11:12:48 +02:00
2024-06-21 14:38:16 +02:00
2024-06-27 13:49:08 +02:00
2024-06-12 11:11:53 +02:00
2024-04-13 13:07:30 +02:00
2023-11-20 11:59:35 +01:00
2023-08-15 15:26:18 -07:00
2024-06-21 14:38:33 +02:00
2023-05-31 09:50:17 +01:00
2022-10-12 17:50:37 -07:00
2024-05-17 12:02:07 +02:00
2024-06-12 11:11:42 +02:00
2023-11-20 11:58:56 +01:00
2024-05-17 12:02:20 +02:00
2024-06-27 13:49:06 +02:00
2022-09-29 07:18:00 +02:00