Re: [iproute PATCH 06/51] iplink_vrf: Complain if main table is not found

2017-08-13 Thread David Ahern
On 8/12/17 6:04 AM, Phil Sutter wrote:
> Signed-off-by: Phil Sutter 
> ---
>  ip/iplink_vrf.c | 5 -
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/ip/iplink_vrf.c b/ip/iplink_vrf.c
> index 917630e853375..809eda5de8f6e 100644
> --- a/ip/iplink_vrf.c
> +++ b/ip/iplink_vrf.c
> @@ -131,7 +131,10 @@ __u32 ipvrf_get_table(const char *name)
>  , sizeof(answer)) < 0) {
>   /* special case "default" vrf to be the main table */
>   if (errno == ENODEV && !strcmp(name, "default"))
> - rtnl_rttable_a2n(_id, "main");
> + if (rtnl_rttable_a2n(_id, "main"))
> + fprintf(stderr,
> + "BUG: RTTable \"main\" not found.\n");
> +
>  
>   return tb_id;
>   }
> 

Acked-by: David Ahern 


[iproute PATCH 06/51] iplink_vrf: Complain if main table is not found

2017-08-12 Thread Phil Sutter
Signed-off-by: Phil Sutter 
---
 ip/iplink_vrf.c | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/ip/iplink_vrf.c b/ip/iplink_vrf.c
index 917630e853375..809eda5de8f6e 100644
--- a/ip/iplink_vrf.c
+++ b/ip/iplink_vrf.c
@@ -131,7 +131,10 @@ __u32 ipvrf_get_table(const char *name)
   , sizeof(answer)) < 0) {
/* special case "default" vrf to be the main table */
if (errno == ENODEV && !strcmp(name, "default"))
-   rtnl_rttable_a2n(_id, "main");
+   if (rtnl_rttable_a2n(_id, "main"))
+   fprintf(stderr,
+   "BUG: RTTable \"main\" not found.\n");
+
 
return tb_id;
}
-- 
2.13.1