netdev - Re: [PATCH] route: add more relaxed option for secure

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 11 Nov 2011 20:33:21 -0500 (EST)
From:	David Miller <davem@...emloft.net>
To:	fbl@...hat.com
Cc:	netdev@...r.kernel.org
Subject: Re: [PATCH] route: add more relaxed option for secure_redirects

From: Flavio Leitner <fbl@...hat.com>
Date: Mon,  7 Nov 2011 22:03:50 -0200

> When the host uses a gateway IP address that is actually an alias
> address, the ICMP redirect message source address can be the gateway's
> main IP address, so the message is ignored by the host regardless of
> the secure_redirects setup.
> 
> The new value (2) allows that ICMP message to be processed.
> The possible values are:
> 
>  0 - Accept ICMP redirect messages only if its source address is the
>      previous gateway address.
>  1 - The same as above. However, if shared_media is FALSE, it has to
>      be for gateways listed in default gateway list as well.
>  2 - Accept ICMP redirects messages ignoring the conditions above.
>  default value is 1.
> 
> Signed-off-by: Flavio Leitner <fbl@...hat.com>

The more I look at this the less I like it.

Look, if IPVS or whatever is translating addresses and this is what
causes the problem then this entity can very well translate the damn
addresses right back in the redirect so it looks legitimate to the
sender.

You can't translate people's addresses, and them let them see that
intenal remapping in ICMP errors.  The redirect is dropped by the
sender because it not only looks like crap, it is crap.

This is fundamentally not the correct way to handle this.

I'm not applying this patch.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html