commit 2bff72e0cf091f4d71af0c81cfc74e8ebd8c6644
author Sreeram Ramachandran <sreeram@google.com> Fri Jul 18 13:03:47 2014 -0700
committer Lorenzo Colitti <lorenzo@google.com> Tue Jul 29 15:09:46 2014 +0900
tree 4a9b6ee2eec871b1b9edb3de92921516b27a0251
parent 060d855a00cb94e8903fd1551c9bf040f42b57d1

Add "scope link" when adding routes without a nexthop.

This is consistent with what /sbin/ip does, and it makes
recursive lookups on secondary tables work even when the main
table is empty.

This was originally proposed by Sreeram as http://ag/506456 to
fix a VPN issue, but then abandoned because it did not fix that
particular problem.

Bug: 16628572
Change-Id: I85753389c683ae2127b21af722463a35a33b60eb

server/RouteController.cpp

diff --git a/server/RouteController.cpp b/server/RouteController.cpp
index 355326d..943a4c1 100644
--- a/server/RouteController.cpp
+++ b/server/RouteController.cpp
@@ -388,6 +388,7 @@
         .rtm_type = type,
         .rtm_family = family,
         .rtm_dst_len = prefixLength,
+        .rtm_scope = static_cast<uint8_t>(nexthop ? RT_SCOPE_UNIVERSE : RT_SCOPE_LINK),
     };
 
     rtattr rtaDst     = { U16_RTA_LENGTH(rawLength), RTA_DST };