- dtucker@cvs.openbsd.org 2014/01/19 11:21:51

     [addrmatch.c]
     Cast the sizeof to socklen_t so it'll work even if the supplied len is
     negative.  Suggested by and ok djm, ok deraadt.

diff --git a/ChangeLog b/ChangeLog
index 9907394684d9b25649cecab873fe42270c212b0c..22578ab21395195078d164e48515087b7af10591 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -16,6 +16,10 @@
    - djm@cvs.openbsd.org 2014/01/19 04:48:08
      [ssh_config.5]
      fix inverted meaning of 'no' and 'yes' for CanonicalizeFallbackLocal
+   - dtucker@cvs.openbsd.org 2014/01/19 11:21:51
+     [addrmatch.c]
+     Cast the sizeof to socklen_t so it'll work even if the supplied len is
+     negative.  Suggested by and ok djm, ok deraadt.
 
 20140118
  - (dtucker) [uidswap.c] Prevent unused variable warnings on Cygwin.  Patch

diff --git a/addrmatch.c b/addrmatch.c
index 649de4139be0e3d8a0703f149d669b91306a7e7b..c443146323dc7c9966c3731fae3c3c3bb9642845 100644 (file)
--- a/addrmatch.c
+++ b/addrmatch.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: addrmatch.c,v 1.8 2014/01/19 04:17:29 dtucker Exp $ */
+/*     $OpenBSD: addrmatch.c,v 1.9 2014/01/19 11:21:51 dtucker Exp $ */
 
 /*
  * Copyright (c) 2004-2008 Damien Miller <djm@mindrot.org>
@@ -88,13 +88,13 @@ addr_sa_to_xaddr(struct sockaddr *sa, socklen_t slen, struct xaddr *xa)
 
        switch (sa->sa_family) {
        case AF_INET:
-               if ((size_t)slen < sizeof(*in4))
+               if (slen < (socklen_t)sizeof(*in4))
                        return -1;
                xa->af = AF_INET;
                memcpy(&xa->v4, &in4->sin_addr, sizeof(xa->v4));
                break;
        case AF_INET6:
-               if ((size_t)slen < sizeof(*in6))
+               if (slen < (socklen_t)sizeof(*in6))
                        return -1;
                xa->af = AF_INET6;
                memcpy(&xa->v6, &in6->sin6_addr, sizeof(xa->v6));