Re: Magic numbers in B::Concise
[p5sagit/p5-mst-13.2.git] / util.c
diff --git a/util.c b/util.c
index 409b1b7..6a0ff44 100644 (file)
--- a/util.c
+++ b/util.c
 #  include <sys/wait.h>
 #endif
 
+#ifdef HAS_SELECT
+# ifdef I_SYS_SELECT
+#  include <sys/select.h>
+# endif
+#endif
+
 #define FLUSH
 
 #ifdef LEAKTEST
@@ -482,6 +488,8 @@ Perl_rninstr(pTHX_ register const char *big, const char *bigend, const char *lit
    If FBMcf_TAIL, the table is created as if the string has a trailing \n. */
 
 /*
+=head1 Miscellaneous Functions
+
 =for apidoc fbm_compile
 
 Analyses the string in order to make fast searches on it using fbm_instr()
@@ -871,6 +879,8 @@ Perl_ibcmp_locale(pTHX_ const char *s1, const char *s2, register I32 len)
 /* copy a string to a safe spot */
 
 /*
+=head1 Memory Management
+
 =for apidoc savepv
 
 Copy a string to a safe spot.  This does not use an SV.
@@ -949,6 +959,7 @@ Perl_form_nocontext(const char* pat, ...)
 #endif /* PERL_IMPLICIT_CONTEXT */
 
 /*
+=head1 Miscellaneous Functions
 =for apidoc form
 
 Takes a sprintf-style format pattern and conventional
@@ -1279,6 +1290,8 @@ Perl_croak_nocontext(const char *pat, ...)
 #endif /* PERL_IMPLICIT_CONTEXT */
 
 /*
+=head1 Warning and Dieing
+
 =for apidoc croak
 
 This is the XSUB-writer's interface to Perl's C<die> function.
@@ -3753,6 +3766,8 @@ return FALSE
         (dp->d_name[1] == '.' && dp->d_name[2] == '\0')))
 
 /*
+=head1 Miscellaneous Functions
+
 =for apidoc getcwd_sv
 
 Fill the sv with current working directory
@@ -3915,13 +3930,15 @@ Perl_getcwd_sv(pTHX_ register SV *sv)
 }
 
 /*
+=head1 SV Manipulation Functions
+
 =for apidoc new_vstring
 
 Returns a pointer to the next character after the parsed
 vstring, as well as updating the passed in sv.
- *
+
 Function must be called like
-       
+
         sv = NEWSV(92,5);
        s = new_vstring(s,sv);
 
@@ -3989,25 +4006,26 @@ Perl_new_vstring(pTHX_ char *s, SV *sv)
 #if !defined(HAS_SOCKETPAIR) && defined(HAS_SOCKET)
 static int
 S_socketpair_udp (int fd[2]) {
+    dTHX;
     /* Fake a datagram socketpair using UDP to localhost.  */
     int sockets[2] = {-1, -1};
     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));
     i = 1;
     do {
-        sockets[i] = socket (AF_INET, SOCK_DGRAM, 0);
+        sockets[i] = PerlSock_socket (AF_INET, SOCK_DGRAM, PF_INET);
         if (sockets[i] == -1)
             goto tidy_up_and_fail;
 
         addresses[i].sin_family = AF_INET;
         addresses[i].sin_addr.s_addr = htonl (INADDR_LOOPBACK);
         addresses[i].sin_port = 0;     /* kernel choses port.  */
-        if (bind (sockets[i], (struct sockaddr *) &addresses[i],
+        if (PerlSock_bind (sockets[i], (struct sockaddr *) &addresses[i],
                   sizeof (struct sockaddr_in))
             == -1)
             goto tidy_up_and_fail;
@@ -4017,13 +4035,13 @@ S_socketpair_udp (int fd[2]) {
        for each connect the other socket to it.  */
     i = 1;
     do {
-        if (getsockname (sockets[i], (struct sockaddr *) &addresses[i], &size)
+        if (PerlSock_getsockname (sockets[i], (struct sockaddr *) &addresses[i], &size)
             == -1)
             goto tidy_up_and_fail;
         if (size != sizeof (struct sockaddr_in))
             goto abort_tidy_up_and_fail;
         /* !1 is 0, !0 is 1 */
-        if (connect(sockets[!i], (struct sockaddr *) &addresses[i],
+        if (PerlSock_connect(sockets[!i], (struct sockaddr *) &addresses[i],
                     sizeof (struct sockaddr_in)) == -1)
             goto tidy_up_and_fail;
     } while (i--);
@@ -4037,7 +4055,7 @@ S_socketpair_udp (int fd[2]) {
            (Who knows if someone somewhere has sin_port as a bitfield and needs
            this routine. (I'm assuming crays have socketpair)) */
         port = addresses[i].sin_port;
-        got = write (sockets[i], &port, sizeof(port));
+        got = PerlLIO_write (sockets[i], &port, sizeof(port));
         if (got != sizeof(port)) {
             if (got == -1)
                 goto tidy_up_and_fail;
@@ -4064,7 +4082,7 @@ S_socketpair_udp (int fd[2]) {
         FD_SET (sockets[0], &rset);
         FD_SET (sockets[1], &rset);
 
-        got = select (max + 1, &rset, NULL, NULL, &waitfor);
+        got = PerlSock_select (max + 1, &rset, NULL, NULL, &waitfor);
         if (got != 2 || !FD_ISSET (sockets[0], &rset)
             || !FD_ISSET (sockets[1], &rset)) {
              /* I hope this is portable and appropriate.  */
@@ -4078,24 +4096,26 @@ 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 {
-            got = recvfrom (sockets[i], (char *) &buffer, sizeof(buffer),
 #ifdef MSG_DONTWAIT
+            got = PerlSock_recvfrom (sockets[i], (char *) &buffer, sizeof(buffer),
                             MSG_DONTWAIT,
+                            (struct sockaddr *) &readfrom, &size);
 #else
+            got = PerlSock_recvfrom (sockets[i], (char *) &buffer, sizeof(buffer),
                             0,
-#endif
                             (struct sockaddr *) &readfrom, &size);
+#endif
 
             if (got == -1)
                     goto tidy_up_and_fail;
             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
@@ -4116,9 +4136,9 @@ S_socketpair_udp (int fd[2]) {
     {
         int save_errno = errno;
         if (sockets[0] != -1)
-            close (sockets[0]);
+            PerlLIO_close (sockets[0]);
         if (sockets[1] != -1)
-            close (sockets[1]);
+            PerlLIO_close (sockets[1]);
         errno = save_errno;
         return -1;
     }
@@ -4127,7 +4147,8 @@ S_socketpair_udp (int fd[2]) {
 int
 Perl_my_socketpair (int family, int type, int protocol, int fd[2]) {
     /* Stevens says that family must be AF_LOCAL, protocol 0.
-       I'm going to enforce that, then ignore it, and use TCP.  */
+       I'm going to enforce that, then ignore it, and use TCP (or UDP).  */
+    dTHX;
     int listener = -1;
     int connector = -1;
     int acceptor = -1;
@@ -4143,48 +4164,50 @@ Perl_my_socketpair (int family, int type, int protocol, int fd[2]) {
         errno = EAFNOSUPPORT;
         return -1;
     }
-    if (!fd)
-        return EINVAL;
+    if (!fd) {
+        errno = EINVAL;
+        return -1;
+    }
 
     if (type == SOCK_DGRAM)
         return S_socketpair_udp (fd);
 
-    listener = socket (AF_INET, type, 0);
+    listener = PerlSock_socket (AF_INET, type, 0);
     if (listener == -1)
         return -1;
     memset (&listen_addr, 0, sizeof (listen_addr));
     listen_addr.sin_family = AF_INET;
     listen_addr.sin_addr.s_addr = htonl (INADDR_LOOPBACK);
     listen_addr.sin_port = 0;  /* kernel choses port.  */
-    if (bind (listener, (struct sockaddr *) &listen_addr, sizeof (listen_addr))
+    if (PerlSock_bind (listener, (struct sockaddr *) &listen_addr, sizeof (listen_addr))
         == -1)
         goto tidy_up_and_fail;
-    if (listen(listener, 1) == -1)
+    if (PerlSock_listen(listener, 1) == -1)
         goto tidy_up_and_fail;
 
-    connector = socket (AF_INET, type, 0);
+    connector = PerlSock_socket (AF_INET, type, 0);
     if (connector == -1)
         goto tidy_up_and_fail;
     /* We want to find out the port number to connect to.  */
     size = sizeof (connect_addr);
-    if (getsockname (listener, (struct sockaddr *) &connect_addr, &size) == -1)
+    if (PerlSock_getsockname (listener, (struct sockaddr *) &connect_addr, &size) == -1)
         goto tidy_up_and_fail;
     if (size != sizeof (connect_addr))
         goto abort_tidy_up_and_fail;
-    if (connect(connector, (struct sockaddr *) &connect_addr,
+    if (PerlSock_connect(connector, (struct sockaddr *) &connect_addr,
                 sizeof (connect_addr)) == -1)
         goto tidy_up_and_fail;
 
     size = sizeof (listen_addr);
-    acceptor = accept (listener, (struct sockaddr *) &listen_addr, &size);
+    acceptor = PerlSock_accept (listener, (struct sockaddr *) &listen_addr, &size);
     if (acceptor == -1)
         goto tidy_up_and_fail;
     if (size != sizeof (listen_addr))
         goto abort_tidy_up_and_fail;
-    close (listener);
+    PerlLIO_close (listener);
     /* Now check we are talking to ourself by matching port and host on the
        two sockets.  */
-    if (getsockname (connector, (struct sockaddr *) &connect_addr, &size) == -1)
+    if (PerlSock_getsockname (connector, (struct sockaddr *) &connect_addr, &size) == -1)
         goto tidy_up_and_fail;
     if (size != sizeof (connect_addr)
         || listen_addr.sin_family != connect_addr.sin_family
@@ -4202,13 +4225,22 @@ Perl_my_socketpair (int family, int type, int protocol, int fd[2]) {
     {
         int save_errno = errno;
         if (listener != -1)
-            close (listener);
+            PerlLIO_close (listener);
         if (connector != -1)
-            close (connector);
+            PerlLIO_close (connector);
         if (acceptor != -1)
-            close (acceptor);
+            PerlLIO_close (acceptor);
         errno = save_errno;
         return -1;
     }
 }
 #endif /* !defined(HAS_SOCKETPAIR) && defined(HAS_SOCKET) */
+#ifdef HAS_SOCKETPAIR
+/* In any case have a stub so that there's code corresponding
+ * to the my_socketpair in global.sym. */
+int
+Perl_my_socketpair (int family, int type, int protocol, int fd[2]) {
+    return socketpair(family, type, protocol, fd);
+}
+#endif
+