From f210dcadf4a921f8ca3718c0b972cee153793488 Mon Sep 17 00:00:00 2001 From: Wayne Davison Date: Mon, 16 Oct 2006 18:11:24 +0000 Subject: [PATCH] - Fixed a compile problem ("len" not defined) in inet_ntop4(). - Replaced sprintf() with snprintf() in inet_ntop6(). --- lib/inet_ntop.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/lib/inet_ntop.c b/lib/inet_ntop.c index 15e3ebd3..7be7368e 100644 --- a/lib/inet_ntop.c +++ b/lib/inet_ntop.c @@ -75,9 +75,10 @@ inet_ntop4(const unsigned char *src, char *dst, size_t size) { static const char *fmt = "%u.%u.%u.%u"; char tmp[sizeof "255.255.255.255"]; + size_t len; - if ((size_t)sprintf(tmp, fmt, src[0], src[1], src[2], src[3]) >= size) - { + len = snprintf(tmp, sizeof tmp, fmt, src[0], src[1], src[2], src[3]); + if (len >= size) { errno = ENOSPC; return (NULL); } @@ -106,7 +107,7 @@ inet_ntop6(const unsigned char *src, char *dst, size_t size) char tmp[sizeof "ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255"], *tp; struct { int base, len; } best, cur; unsigned int words[NS_IN6ADDRSZ / NS_INT16SZ]; - int i; + int i, inc; /* * Preprocess: @@ -157,13 +158,14 @@ inet_ntop6(const unsigned char *src, char *dst, size_t size) /* Is this address an encapsulated IPv4? */ if (i == 6 && best.base == 0 && (best.len == 6 || (best.len == 5 && words[5] == 0xffff))) { - if (!inet_ntop4(src+12, tp, - sizeof tmp - (tp - tmp))) + if (!inet_ntop4(src+12, tp, sizeof tmp - (tp - tmp))) return (NULL); tp += strlen(tp); break; } - tp += sprintf(tp, "%x", words[i]); + inc = snprintf(tp, 5, "%x", words[i]); + assert(inc < 5); + tp += inc; } /* Was it a trailing run of 0x00's? */ if (best.base != -1 && (best.base + best.len) == -- 2.34.1