From: Nicholas Clark <nick@ccl4.org>
Date: Wed, 26 Dec 2001 19:07:33 +0000 (+0000)
Subject: Re: [ PATCH ] Cygwin groups.t  (PLEASE CHECK)
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=ae92b34e9d83c8393b2371f5991925e60ecffa3a;p=p5sagit%2Fp5-mst-13.2.git

Re: [ PATCH ] Cygwin groups.t  (PLEASE CHECK)
Message-ID: <20011226190732.J48842@plum.flirble.org>

socketpair() (un)signedness nit.

p4raw-id: //depot/perl@13893
---

diff --git a/util.c b/util.c
index 409b1b7..6e50628 100644
--- a/util.c
+++ b/util.c
@@ -3994,7 +3994,7 @@ S_socketpair_udp (int fd[2]) {
     struct sockaddr_in addresses[2];
     int i;
     Sock_size_t size = sizeof (struct sockaddr_in);
-    short port;
+    unsigned short port;
     int got;
 
     memset (&addresses, 0, sizeof (addresses));
@@ -4078,7 +4078,7 @@ S_socketpair_udp (int fd[2]) {
        (hence MSG_DONTWAIT). Or that what arrives was sent by us.  */
     {
         struct sockaddr_in readfrom;
-        short buffer[2];
+        unsigned short buffer[2];
 
         i = 1;
         do {
@@ -4095,7 +4095,7 @@ S_socketpair_udp (int fd[2]) {
             if (got != sizeof(port)
                 || size != sizeof (struct sockaddr_in)
                 /* Check other socket sent us its port.  */
-                || buffer[0] != addresses[!i].sin_port
+                || buffer[0] != (unsigned short) addresses[!i].sin_port
                 /* Check kernel says we got the datagram from that socket.  */
                 || readfrom.sin_family != addresses[!i].sin_family
                 || readfrom.sin_addr.s_addr != addresses[!i].sin_addr.s_addr