diff options
| author | Stefan Nuernberger <snu@amazon.com> | 2018-09-17 19:46:53 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-09-18 04:37:46 +0200 |
| commit | 076ed3da0c9b2f88d9157dbe7044a45641ae369e (patch) | |
| tree | f40d60c2bccd9cee2a8b592955b0c1b500c76b01 /drivers/net/ethernet/freescale/gianfar.c | |
| parent | net: socionext: Fix two sleep-in-atomic-context bugs in ave_rxfifo_reset() (diff) | |
| download | linux-076ed3da0c9b2f88d9157dbe7044a45641ae369e.tar.xz linux-076ed3da0c9b2f88d9157dbe7044a45641ae369e.zip | |
net/ipv4: defensive cipso option parsing
commit 40413955ee26 ("Cipso: cipso_v4_optptr enter infinite loop") fixed
a possible infinite loop in the IP option parsing of CIPSO. The fix
assumes that ip_options_compile filtered out all zero length options and
that no other one-byte options beside IPOPT_END and IPOPT_NOOP exist.
While this assumption currently holds true, add explicit checks for zero
length and invalid length options to be safe for the future. Even though
ip_options_compile should have validated the options, the introduction of
new one-byte options can still confuse this code without the additional
checks.
Signed-off-by: Stefan Nuernberger <snu@amazon.com>
Cc: David Woodhouse <dwmw@amazon.co.uk>
Cc: Simon Veith <sveith@amazon.de>
Cc: stable@vger.kernel.org
Acked-by: Paul Moore <paul@paul-moore.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/freescale/gianfar.c')
0 files changed, 0 insertions, 0 deletions