diff options
| author | Julian Anastasov <ja@ssi.bg> | 2026-09-11 14:43:15 +0300 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2026-09-18 10:58:07 +0200 |
| commit | e290145564886d6a3038810c621f738c1fe9fa51 (patch) | |
| tree | bc51644ca1ae63a18da1238d68f51d67aa5028a8 /tools/perf/scripts/python/stackcollapse.py | |
| parent | a311a898172743558b82f6035ef2aa8c310a4223 (diff) | |
| download | linux-e290145564886d6a3038810c621f738c1fe9fa51.tar.gz linux-e290145564886d6a3038810c621f738c1fe9fa51.zip | |
ipvs: revalidate ihl before icmp_send
While the outer IP header is already pulled into the skb head, we must
be careful and revalidate the embedded headers after reading them from
the skb frags to prevent possible out-of-bounds access.
One such place reported by Sashiko is ip_vs_in_icmp() where local
process can change the ihl field and after pskb_may_pull() we can see
larger value. Even if icmp_send() has checks to prevent out-of-bounds
access, play safe and add check to drop the packet if the ihl field is
changed. As the outer headers are pulled, make sure the transport
header is updated too, it was used before commit 7fcc2fe39fed ("net:
icmp: avoid invalid transport header access in icmp_send tracepoint")
Fixes: f2edb9f7706d ("ipvs: implement passive PMTUD for IPIP packets")
Link: https://sashiko.dev/#/patchset/20260806105211.34622-1-ja%40ssi.bg
Signed-off-by: Julian Anastasov <ja@ssi.bg>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions
