krb5 commit: Correct includes for unlockiter.c

Tom Yu tlyu at MIT.EDU
Mon Aug 4 13:58:48 EDT 2014


https://github.com/krb5/krb5/commit/f85675510f9028879228193bae25f1b127134fc4
commit f85675510f9028879228193bae25f1b127134fc4
Author: Tom Yu <tlyu at mit.edu>
Date:   Mon Aug 4 12:34:24 2014 -0400

    Correct includes for unlockiter.c
    
    Some platforms (e.g., Solaris) need a declaration of memset() for the
    FD_ZERO() macro to work, contrary to POSIX standards.  Add an
    inclusion of <string.h> to accommodate them.  Also add <sys/time.h>,
    possibly needed by some older platforms, and remove a spurious
    inclusion of <sys/socket.h>.
    
    ticket: 7977

 src/tests/unlockiter.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/src/tests/unlockiter.c b/src/tests/unlockiter.c
index e2f2eb5..b0c48f9 100644
--- a/src/tests/unlockiter.c
+++ b/src/tests/unlockiter.c
@@ -35,7 +35,8 @@
  */
 
 #include <sys/types.h>
-#include <sys/socket.h>
+#include <sys/select.h>
+#include <sys/time.h>
 #include <sys/wait.h>
 
 #include <errno.h>
@@ -43,6 +44,7 @@
 #include <kadm5/admin.h>
 #include <signal.h>
 #include <stdlib.h>
+#include <string.h>             /* Some platforms need memset() for FD_ZERO */
 #include <unistd.h>
 
 struct cb_arg {


More information about the cvs-krb5 mailing list