Better handling of inet_ntop() failures (which should not happen in practice, since...

[BearSSL] / tools / client.c

diff --git a/tools/client.c b/tools/client.c
index 692b60f..7acfebf 100644 (file)
--- a/tools/client.c
+++ b/tools/client.c
@@ -70,7 +70,9 @@ host_connect(const char *host, const char *port, int verbose)
                        addr = NULL;
                }
                if (addr != NULL) {
                        addr = NULL;
                }
                if (addr != NULL) {

-                       inet_ntop(p->ai_family, addr, tmp, sizeof tmp);
+                       if (!inet_ntop(p->ai_family, addr, tmp, sizeof tmp)) {
+                               strcpy(tmp, "<invalid>");
+                       }

                } else {
                        sprintf(tmp, "<unknown family: %d>",
                                (int)sa->sa_family);
                } else {
                        sprintf(tmp, "<unknown family: %d>",
                                (int)sa->sa_family);