[PATCH iproute2-next v2 2/2] gre/gre6: Unify attribute addition to netlink buffer

[Serhey Popovych]

There are couple of minor improvements:

  1) Check erspan_ver == 2 in gre6. It still could
     be 1 if erspan_idx is 0.

  2) Add tunnel encapsulation attributes only when
     collect metadata not in effect in gre.

  3) Trivial: address checkpatch issues.

Signed-off-by: Serhey Popovych <serhe.popov...@gmail.com>
---
 ip/link_gre.c  |   23 ++++++++++++-----------
 ip/link_gre6.c |   13 ++++++++-----
 2 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/ip/link_gre.c b/ip/link_gre.c
index 8eb60d1..69bb88e 100644
--- a/ip/link_gre.c
+++ b/ip/link_gre.c
@@ -366,6 +366,8 @@ get_failed:
                addattr_l(n, 1024, IFLA_GRE_LOCAL, &saddr, 4);
                addattr_l(n, 1024, IFLA_GRE_REMOTE, &daddr, 4);
                addattr_l(n, 1024, IFLA_GRE_PMTUDISC, &pmtudisc, 1);
+               if (ignore_df)
+                       addattr8(n, 1024, IFLA_GRE_IGNORE_DF, ignore_df & 1);
                if (link)
                        addattr32(n, 1024, IFLA_GRE_LINK, link);
                addattr_l(n, 1024, IFLA_GRE_TTL, &ttl, 1);
@@ -374,24 +376,23 @@ get_failed:
                if (erspan_ver) {
                        addattr8(n, 1024, IFLA_GRE_ERSPAN_VER, erspan_ver);
                        if (erspan_ver == 1 && erspan_idx != 0) {
-                               addattr32(n, 1024, IFLA_GRE_ERSPAN_INDEX, 
erspan_idx);
+                               addattr32(n, 1024,
+                                         IFLA_GRE_ERSPAN_INDEX, erspan_idx);
                        } else if (erspan_ver == 2) {
-                               addattr8(n, 1024, IFLA_GRE_ERSPAN_DIR, 
erspan_dir);
-                               addattr16(n, 1024, IFLA_GRE_ERSPAN_HWID, 
erspan_hwid);
+                               addattr8(n, 1024,
+                                        IFLA_GRE_ERSPAN_DIR, erspan_dir);
+                               addattr16(n, 1024,
+                                         IFLA_GRE_ERSPAN_HWID, erspan_hwid);
                        }
                }
+               addattr16(n, 1024, IFLA_GRE_ENCAP_TYPE, encaptype);
+               addattr16(n, 1024, IFLA_GRE_ENCAP_FLAGS, encapflags);
+               addattr16(n, 1024, IFLA_GRE_ENCAP_SPORT, htons(encapsport));
+               addattr16(n, 1024, IFLA_GRE_ENCAP_DPORT, htons(encapdport));
        } else {
                addattr_l(n, 1024, IFLA_GRE_COLLECT_METADATA, NULL, 0);
        }
 
-       addattr16(n, 1024, IFLA_GRE_ENCAP_TYPE, encaptype);
-       addattr16(n, 1024, IFLA_GRE_ENCAP_FLAGS, encapflags);
-       addattr16(n, 1024, IFLA_GRE_ENCAP_SPORT, htons(encapsport));
-       addattr16(n, 1024, IFLA_GRE_ENCAP_DPORT, htons(encapdport));
-
-       if (ignore_df)
-               addattr8(n, 1024, IFLA_GRE_IGNORE_DF, ignore_df & 1);
-
        return 0;
 }
 
diff --git a/ip/link_gre6.c b/ip/link_gre6.c
index 60c0b54..3e80625 100644
--- a/ip/link_gre6.c
+++ b/ip/link_gre6.c
@@ -405,10 +405,13 @@ get_failed:
                if (erspan_ver) {
                        addattr8(n, 1024, IFLA_GRE_ERSPAN_VER, erspan_ver);
                        if (erspan_ver == 1 && erspan_idx != 0) {
-                               addattr32(n, 1024, IFLA_GRE_ERSPAN_INDEX, 
erspan_idx);
-                       } else {
-                               addattr8(n, 1024, IFLA_GRE_ERSPAN_DIR, 
erspan_dir);
-                               addattr16(n, 1024, IFLA_GRE_ERSPAN_HWID, 
erspan_hwid);
+                               addattr32(n, 1024,
+                                         IFLA_GRE_ERSPAN_INDEX, erspan_idx);
+                       } else if (erspan_ver == 2) {
+                               addattr8(n, 1024,
+                                        IFLA_GRE_ERSPAN_DIR, erspan_dir);
+                               addattr16(n, 1024,
+                                         IFLA_GRE_ERSPAN_HWID, erspan_hwid);
                        }
                }
                addattr16(n, 1024, IFLA_GRE_ENCAP_TYPE, encaptype);
@@ -645,7 +648,7 @@ static void gre_print_opt(struct link_util *lu, FILE *f, 
struct rtattr *tb[])
 }
 
 static void gre_print_help(struct link_util *lu, int argc, char **argv,
-       FILE *f)
+                          FILE *f)
 {
        print_usage(f);
 }
-- 
1.7.10.4