Fix if_linux_ipv6 verbose output of interface addresses

Previously the verbose output of if_linux_ipv6_open looked like this:

    found interface ab c:  ab: a b: abc:   : a  🔡     scope 0

This changes the output to:

    found interface eth0 inet6 ab0c🆎a0b🔤0:a00🔡0 scope 0

Signed-off-by: Orivej Desh <[email protected]>

Author: orivej

opal/mca/if/linux_ipv6/if_linux_ipv6.c

@@ -77,7 +77,21 @@ opal_if_base_component_t mca_if_linux_ipv6_component = {
     },
 };
 
-/* configure using getifaddrs(3) */
+static int hex2int(char hex)
+{
+    if ('0' <= hex && hex <= '9') return hex - '0';
+    if ('A' <= hex && hex <= 'F') return hex - 'A' + 10;
+    if ('a' <= hex && hex <= 'f') return hex - 'a' + 10;
+    abort();
+}
+
+static void hexdecode(const char *src, char *dst, size_t dstsize)
+{
+    for (size_t i = 0; i < dstsize; i++) {
+        dst[i] = hex2int(src[i * 2]) * 16 + hex2int(src[i * 2 + 1]);
+    }
+}
+
 static int if_linux_ipv6_open(void)
 {
 #if OPAL_ENABLE_IPV6
@@ -86,33 +100,26 @@ static int if_linux_ipv6_open(void)
         char ifname[IF_NAMESIZE];
         unsigned int idx, pfxlen, scope, dadstat;
         struct in6_addr a6;
-        int iter;
         uint32_t flag;
-        unsigned int addrbyte[16];
+        char addrhex[sizeof a6.s6_addr * 2 + 1];
+        char addrstr[INET6_ADDRSTRLEN];
 
-        while (fscanf(f, "%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x %x %x %x %x %20s\n",
-                      &addrbyte[0], &addrbyte[1], &addrbyte[2], &addrbyte[3],
-                      &addrbyte[4], &addrbyte[5], &addrbyte[6], &addrbyte[7],
-                      &addrbyte[8], &addrbyte[9], &addrbyte[10], &addrbyte[11],
-                      &addrbyte[12], &addrbyte[13], &addrbyte[14], &addrbyte[15],
+        while (fscanf(f, "%s %x %x %x %x %s\n", addrhex,
                       &idx, &pfxlen, &scope, &dadstat, ifname) != EOF) {
             opal_if_t *intf;
 
+            hexdecode(addrhex, a6.s6_addr, sizeof a6.s6_addr);
+            inet_ntop(AF_INET6, a6.s6_addr, addrstr, sizeof addrstr);
+
             opal_output_verbose(1, opal_if_base_framework.framework_output,
-                                "found interface %2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x scope %x\n",
-                                addrbyte[0], addrbyte[1], addrbyte[2], addrbyte[3],
-                                addrbyte[4], addrbyte[5], addrbyte[6], addrbyte[7],
-                                addrbyte[8], addrbyte[9], addrbyte[10], addrbyte[11],
-                                addrbyte[12], addrbyte[13], addrbyte[14], addrbyte[15], scope);
+                                "found interface %s inet6 %s scope %x\n",
+                                ifname, addrstr, scope);
 
             /* Only interested in global (0x00) scope */
             if (scope != 0x00)  {
                 opal_output_verbose(1, opal_if_base_framework.framework_output,
-                                    "skipping interface %2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x scope %x\n",
-                                    addrbyte[0], addrbyte[1], addrbyte[2], addrbyte[3],
-                                    addrbyte[4], addrbyte[5], addrbyte[6], addrbyte[7],
-                                    addrbyte[8], addrbyte[9], addrbyte[10], addrbyte[11],
-                                    addrbyte[12], addrbyte[13], addrbyte[14], addrbyte[15], scope);
+                                    "skipped interface %s inet6 %s scope %x\n",
+                                    ifname, addrstr, scope);
                 continue;
             }
 
@@ -125,10 +132,6 @@ static int if_linux_ipv6_open(void)
             }
             intf->af_family = AF_INET6;
 
-            for (iter = 0; iter < 16; iter++) {
-                a6.s6_addr[iter] = addrbyte[iter];
-            }
-
             /* now construct the opal_if_t */
             opal_string_copy(intf->if_name, ifname, IF_NAMESIZE);
             intf->if_index = opal_list_get_size(&opal_if_list)+1;
@@ -147,17 +150,12 @@ static int if_linux_ipv6_open(void)
                to list */
             opal_list_append(&opal_if_list, &(intf->super));
             opal_output_verbose(1, opal_if_base_framework.framework_output,
-                                "added interface %2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x:%2x%2x\n",
-                                addrbyte[0], addrbyte[1], addrbyte[2], addrbyte[3],
-                                addrbyte[4], addrbyte[5], addrbyte[6], addrbyte[7],
-                                addrbyte[8], addrbyte[9], addrbyte[10], addrbyte[11],
-                                addrbyte[12], addrbyte[13], addrbyte[14], addrbyte[15]);
+                                "added interface %s inet6 %s scope %x\n",
+                                ifname, addrstr, scope);
         } /* of while */
         fclose(f);
     }
 #endif  /* OPAL_ENABLE_IPV6 */
 
     return OPAL_SUCCESS;
 }
-
-