Move include of sys/types.h in kopenafs.c
authorAntoine Verheijen <antoine@ualberta.ca>
Fri, 3 Dec 2010 07:26:24 +0000 (00:26 -0700)
committerDerrick Brashear <shadow@dementia.org>
Fri, 3 Dec 2010 19:27:54 +0000 (11:27 -0800)
The netinet/in.h header file requires the inclusion of sys/types.h
(or some portion thereof). Most systems include it implicitly
within netinet/in.h. Some, such as OpenBSD, do not. In kopenafs.c,
sys/types.h is included after netinet/in.h which is fine in most
cases but means that, as a result, it will not compile on OpenBSD.
Moving sys/types.h before netinet/in.h solves this problem (and
should cause no problems on any other system).

Change-Id: Ifc3295f3eb4f0b2b4718d0b2aa6dcbb2b06da8df
Reviewed-on: http://gerrit.openafs.org/3429
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

src/kopenafs/kopenafs.c

index 13f66b2..9d987a6 100644 (file)
@@ -13,6 +13,7 @@
  */
 
 #include <errno.h>
+#include <sys/types.h>
 #include <netinet/in.h>
 #include <signal.h>
 #include <stdlib.h>
@@ -23,7 +24,6 @@
 # endif
 #endif
 #include <sys/param.h>
-#include <sys/types.h>
 #include <unistd.h>
 
 #include <afsconfig.h>