All tunnels already support for parsing/adding zero
endpoints and vti6 isn't an exception.

This check was added as part of commit 2a80154fde40
(vti6: fix local/remote any addr handling) and looks
too restrictive as purpose of change is to avoid
endpoint configuration from uninitialized data.

Signed-off-by: Serhey Popovych <serhe.popov...@gmail.com>
---
 ip/link_vti6.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/ip/link_vti6.c b/ip/link_vti6.c
index f631839..4136b0e 100644
--- a/ip/link_vti6.c
+++ b/ip/link_vti6.c
@@ -154,10 +154,8 @@ get_failed:
        addattr32(n, 1024, IFLA_VTI_IKEY, ikey);
        addattr32(n, 1024, IFLA_VTI_OKEY, okey);
 
-       if (memcmp(&saddr, &in6addr_any, sizeof(in6addr_any)))
-           addattr_l(n, 1024, IFLA_VTI_LOCAL, &saddr, sizeof(saddr));
-       if (memcmp(&daddr, &in6addr_any, sizeof(in6addr_any)))
-           addattr_l(n, 1024, IFLA_VTI_REMOTE, &daddr, sizeof(daddr));
+       addattr_l(n, 1024, IFLA_VTI_LOCAL, &saddr, sizeof(saddr));
+       addattr_l(n, 1024, IFLA_VTI_REMOTE, &daddr, sizeof(daddr));
        addattr32(n, 1024, IFLA_VTI_FWMARK, fwmark);
        if (link)
                addattr32(n, 1024, IFLA_VTI_LINK, link);
-- 
1.7.10.4

Reply via email to