OSSP CVS Repository

ossp - Check-in [4736]
Not logged in 		[Honeypot]  [Browse]  [Home]  [Login]  [Reports] 
[Search]  [Ticket]  [Timeline] 
  [Patchset]  [Tagging/Branching] 

Check-in Number: 4736
Date: 2004-Oct-03 16:49:37 (local)
2004-Oct-03 14:49:37 (UTC)
User:rse
Branch:
Comment: Do not pass NULL directly to variadic functions because NULL is not guarrantied to be a pointer.
Tickets:
Inspections:
Files:
ossp-pkg/sa/ChangeLog      1.41 -> 1.42     4 inserted, 0 deleted
ossp-pkg/sa/sa.c      1.86 -> 1.87     6 inserted, 6 deleted
ossp-pkg/sa/ChangeLog 1.41 -> 1.42 

--- ChangeLog    2004/10/03 14:00:23     1.41
+++ ChangeLog    2004/10/03 14:49:37     1.42
@@ -13,6 +13,10 @@
 
   Changes between 1.2.2 and 1.2.3 (26-Jun-2004 to xx-xxx-2004)
 
+   o Do not pass NULL directly to variadic functions because NULL
+     is not guarrantied to be a pointer.
+     [Ralf S. Engelschall <rse@engelschall.com>]
+
    o Added SA_SYSCALL_GETHOSTBYNAME for supporting the overloading
      of gethostbyname(2) system calls.
      [Ralf S. Engelschall <rse@engelschall.com>]

ossp-pkg/sa/sa.c 1.86 -> 1.87 

--- sa.c 2004/10/03 14:00:23     1.86
+++ sa.c 2004/10/03 14:49:37     1.87
@@ -1517,7 +1517,7 @@
     else
         tv = NULL;
     do {
-        n = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &rset, &wset, NULL, tv);
+        n = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &rset, &wset, (fd_set *)NULL, tv);
     } while (n == -1 && errno == EINTR);
 
     /* decide on return semantic */
@@ -1611,7 +1611,7 @@
         FD_ZERO(&fds);
         FD_SET(sa->fdSocket, &fds);
         do {
-            n = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &fds, NULL, NULL,
+            n = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &fds, (fd_set *)NULL, (fd_set *)NULL,
                              &sa->tvTimeout[SA_TIMEOUT_ACCEPT]);
         } while (n == -1 && errno == EINTR);
         if (n == 0)
@@ -1772,7 +1772,7 @@
         FD_ZERO(&fds);
         FD_SET(sa->fdSocket, &fds);
         do {
-            rv = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &fds, NULL, NULL,
+            rv = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &fds, (fd_set *)NULL, (fd_set *)NULL,
                               &sa->tvTimeout[SA_TIMEOUT_READ]);
         } while (rv == -1 && errno == EINTR);
         if (rv == 0) {
@@ -1951,7 +1951,7 @@
         FD_ZERO(&fds);
         FD_SET(sa->fdSocket, &fds);
         do {
-            rv = SA_SC_CALL_5(sa, select, sa->fdSocket+1, NULL, &fds, NULL,
+            rv = SA_SC_CALL_5(sa, select, sa->fdSocket+1, (fd_set *)NULL, &fds, (fd_set *)NULL,
                               &sa->tvTimeout[SA_TIMEOUT_WRITE]);
         } while (rv == -1 && errno == EINTR);
         if (rv == 0) {
@@ -2196,7 +2196,7 @@
         FD_ZERO(&fds);
         FD_SET(sa->fdSocket, &fds);
         do {
-            k = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &fds, NULL, NULL,
+            k = SA_SC_CALL_5(sa, select, sa->fdSocket+1, &fds, (fd_set *)NULL, (fd_set *)NULL,
                              &sa->tvTimeout[SA_TIMEOUT_READ]);
         } while (k == -1 && errno == EINTR);
         if (k == 0)
@@ -2253,7 +2253,7 @@
         FD_ZERO(&fds);
         FD_SET(sa->fdSocket, &fds);
         do {
-            k = SA_SC_CALL_5(sa, select, sa->fdSocket+1, NULL, &fds, NULL,
+            k = SA_SC_CALL_5(sa, select, sa->fdSocket+1, (fd_set *)NULL, &fds, (fd_set *)NULL,
                              &sa->tvTimeout[SA_TIMEOUT_WRITE]);
         } while (k == -1 && errno == EINTR);
         if (k == 0)

CVSTrac 2.0.1