On Fri, Dec 5, 2014 at 10:48 AM, Kristian Evensen
<kristian.even...@gmail.com> wrote:
>
> From: Kristian Evensen <kristian.even...@gmail.com>
>
> This patch depends on "Pass source address to proto_add_ipv4_route".
>
> I have not found a scenario that would break by setting the source address on
> default, but please let me know if any special considerations should be taken.
>
> Signed-off-by: Kristian Evensen <kristian.even...@gmail.com>
> ---
> package/network/config/netifd/files/lib/netifd/dhcp.script | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/network/config/netifd/files/lib/netifd/dhcp.script
> b/package/network/config/netifd/files/lib/netifd/dhcp.script
> index 90fa6d3..17e22af 100755
> --- a/package/network/config/netifd/files/lib/netifd/dhcp.script
> +++ b/package/network/config/netifd/files/lib/netifd/dhcp.script
> @@ -20,7 +20,7 @@ setup_interface () {
> # TODO: apply $broadcast
>
> for i in $router; do
> - proto_add_ipv4_route 0.0.0.0 0 "$i"
> + proto_add_ipv4_route 0.0.0.0 0 "$i" "$ip"
> done
>
> # CIDR STATIC ROUTES (rfc3442)
> --
> 1.9.1
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
As a side effect of this patch I noticed multiple default routes (one
default route with source IP set and one default route with source IP
unset) are present in netifd if the dhcp server replies the default
route both in the router option and the staticroutes option. Before
the patch the default routes were considered as identical by netifd as
the route key was the same; but since the source IP is set now for one
of the default routes this is not the case anymore.
Is there any reason why the source IP is not set when the staticroutes
are added in set_classless_routes in the dhcp script ?
Hans
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
