Get port ifname once rather than on both sides of if(is_json_context).

Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
---
 bridge/mdb.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/bridge/mdb.c b/bridge/mdb.c
index f38dc67c849a..9bdef0262c54 100644
--- a/bridge/mdb.c
+++ b/bridge/mdb.c
@@ -208,19 +208,19 @@ static void print_router_entries(FILE *fp, struct 
nlmsghdr *n,
        } else {
                struct rtattr *i = RTA_DATA(router);
                uint32_t *port_ifindex = RTA_DATA(i);
+               const char *port_name = ll_index_to_name(*port_ifindex);
 
                if (is_json_context()) {
                        open_json_array(PRINT_JSON, brifname);
                        open_json_object(NULL);
 
                        print_string(PRINT_JSON, "port", NULL,
-                                    ll_index_to_name(*port_ifindex));
+                                    port_name);
                        close_json_object();
                        close_json_array(PRINT_JSON, NULL);
                } else {
                        fprintf(fp, "router port dev %s master %s\n",
-                               ll_index_to_name(*port_ifindex),
-                               brifname);
+                               port_name, brifname);
                }
        }
        close_json_array(PRINT_JSON, NULL);
-- 
2.17.1

Reply via email to