rev 1.3 introduced a check to an if statement without adding braces.
authorjsg <jsg@openbsd.org>
Thu, 14 May 2015 11:52:43 +0000 (11:52 +0000)
committerjsg <jsg@openbsd.org>
Thu, 14 May 2015 11:52:43 +0000 (11:52 +0000)
Claudio points out the size is checked by an earlier test so just
remove it to restore the original handling of the partial octet case.

Discussed with claudio and gilles.

lib/libc/net/inet_net_ntop.c

index 35ae790..652b240 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: inet_net_ntop.c,v 1.7 2012/06/22 19:13:37 gilles Exp $        */
+/*     $OpenBSD: inet_net_ntop.c,v 1.8 2015/05/14 11:52:43 jsg Exp $   */
 
 /*
  * Copyright (c) 2012 by Gilles Chehade <gilles@openbsd.org>
@@ -114,8 +114,6 @@ inet_net_ntop_ipv4(const u_char *src, int bits, char *dst, size_t size)
                if (ep - dst < sizeof ".255")
                        goto emsgsize;
                if (dst != odst)
-                       if (dst + 1 >= ep)
-                               goto emsgsize;
                        *dst++ = '.';
                m = ((1 << b) - 1) << (8 - b);
                advance = snprintf(dst, ep - dst, "%u", *src & m);