[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] net_if: new module
From: |
Eric Blake |
Subject: |
[PATCH] net_if: new module |
Date: |
Wed, 5 Sep 2012 15:59:10 -0600 |
OpenBSD's <net/if.h> is not self-contained; this should compile:
| #define _POSIX_C_SOURCE 200809L
| #include <net/if.h>
| struct if_nameindex i;
but fails with:
In file included from foo.c:2:
/usr/include/net/if.h:112: error: expected specifier-qualifier-list before
'u_int'
...
/usr/include/net/if.h:674: error: field 'dstaddr' has incomplete type
In file included from /usr/include/net/if.h:691,
from foo.c:2:
/usr/include/net/if_arp.h:79: error: field 'arp_pa' has incomplete type
/usr/include/net/if_arp.h:80: error: field 'arp_ha' has incomplete type
*** Error code 1
We already had a test program for systems with the if_* functions
(including OpenBSD); but in adding this module, I had to enhance
that test to also work on platforms like mingw where we are now
providing a new header.
* modules/net_if: New module, borrowing ideas from netinet_in.
* m4/net_if_h.m4: New file.
* lib/net_if.in.h: Likewise.
* doc/posix-headers/net_if.texi (net/if.h): Document it.
* MODULES.html.sh (lacking POSIX:2008): Likewise.
* tests/test-net_if.c: Make function checks conditional.
Reported by Jasper Lievisse Adriaanse <address@hidden>.
Signed-off-by: Eric Blake <address@hidden>
---
ChangeLog | 11 +++++++++
MODULES.html.sh | 1 +
doc/posix-headers/net_if.texi | 5 +++-
lib/net_if.in.h | 53 +++++++++++++++++++++++++++++++++++++++++++
m4/net_if_h.m4 | 31 +++++++++++++++++++++++++
modules/net_if | 49 +++++++++++++++++++++++++++++++++++++++
tests/test-net_if.c | 12 ++++++++--
7 files changed, 159 insertions(+), 3 deletions(-)
create mode 100644 lib/net_if.in.h
create mode 100644 m4/net_if_h.m4
create mode 100644 modules/net_if
diff --git a/ChangeLog b/ChangeLog
index dea1f86..d9f6f89 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2012-09-05 Eric Blake <address@hidden>
+
+ net_if: new module
+ * modules/net_if: New module, borrowing ideas from netinet_in.
+ * m4/net_if_h.m4: New file.
+ * lib/net_if.in.h: Likewise.
+ * doc/posix-headers/net_if.texi (net/if.h): Document it.
+ * MODULES.html.sh (lacking POSIX:2008): Likewise.
+ * tests/test-net_if.c: Make function checks conditional.
+ Reported by Jasper Lievisse Adriaanse <address@hidden>.
+
2012-09-05 Mats Erik Andersson <address@hidden> (tiny change)
readutmp: fix non-portable UT_PID use
diff --git a/MODULES.html.sh b/MODULES.html.sh
index 8ced611..0e34531 100755
--- a/MODULES.html.sh
+++ b/MODULES.html.sh
@@ -2430,6 +2430,7 @@ func_all_modules ()
func_module mkfifo
func_module mknod
func_module mkstemp
+ func_module net_if
func_module netdb
func_module netinet_in
func_module nl_langinfo
diff --git a/doc/posix-headers/net_if.texi b/doc/posix-headers/net_if.texi
index 38a52f4..fa01b6e 100644
--- a/doc/posix-headers/net_if.texi
+++ b/doc/posix-headers/net_if.texi
@@ -3,10 +3,13 @@ net/if.h
POSIX specification:@*
@url{http://www.opengroup.org/onlinepubs/9699919799/basedefs/net_if.h.html}
-Gnulib module: ---
+Gnulib module: net_if
Portability problems fixed by Gnulib:
@itemize
address@hidden
+This header file is not self-contained on some platforms:
+FreeBSD 8.2, OpenBSD 5.1.
@end itemize
Portability problems not fixed by Gnulib:
diff --git a/lib/net_if.in.h b/lib/net_if.in.h
new file mode 100644
index 0000000..24f9f83
--- /dev/null
+++ b/lib/net_if.in.h
@@ -0,0 +1,53 @@
+/* Substitute for <net/if.h>.
+ Copyright (C) 2007-2012 Free Software Foundation, Inc.
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2, or (at your option)
+ any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program; if not, see <http://www.gnu.org/licenses/>. */
+
+#ifndef address@hidden@_NET_IF_H
+
+#if __GNUC__ >= 3
address@hidden@
+#endif
address@hidden@
+
+#if @HAVE_NET_IF_H@
+
+/* On some platforms, <net/if.h> assumes prior inclusion of
+ <sys/socket.h>. */
+# include <sys/socket.h>
+
+/* The include_next requires a split double-inclusion guard. */
+# @INCLUDE_NEXT@ @NEXT_NET_IF_H@
+
+#endif
+
+#ifndef address@hidden@_NET_IF_H
+#define address@hidden@_NET_IF_H
+
+#if address@hidden@
+
+/* A platform that lacks <net/if.h>. */
+
+struct if_nameindex
+{
+ unsigned if_index;
+ char *if_name;
+};
+
+# define IF_NAMESIZE 1 /* Without if_* functions, this supports only "". */
+
+#endif
+
+#endif /* address@hidden@_NET_IF_H */
+#endif /* address@hidden@_NET_IF_H */
diff --git a/m4/net_if_h.m4 b/m4/net_if_h.m4
new file mode 100644
index 0000000..171101f
--- /dev/null
+++ b/m4/net_if_h.m4
@@ -0,0 +1,31 @@
+# net_if_h.m4 serial 1
+dnl Copyright (C) 2006-2012 Free Software Foundation, Inc.
+dnl This file is free software; the Free Software Foundation
+dnl gives unlimited permission to copy and/or distribute it,
+dnl with or without modifications, as long as this notice is preserved.
+
+AC_DEFUN([gl_HEADER_NET_IF],
+[
+ AC_CACHE_CHECK([whether <net/if.h> is self-contained],
+ [gl_cv_header_net_if_h_selfcontained],
+ [
+ AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <net/if.h>]], [[]])],
+ [gl_cv_header_net_if_h_selfcontained=yes],
+ [gl_cv_header_net_if_h_selfcontained=no])
+ ])
+ if test $gl_cv_header_net_if_h_selfcontained = yes; then
+ NET_IF_H=''
+ else
+ NET_IF_H='net/if.h'
+ AC_CHECK_HEADERS([net/if.h], [], [], [[#include <sys/socket.h>]])
+ gl_NEXT_HEADERS([net/if.h])
+ if test $ac_cv_header_net_if_h = yes; then
+ HAVE_NET_IF_H=1
+ else
+ HAVE_NET_IF_H=0
+ fi
+ AC_SUBST([HAVE_NET_IF_H])
+ fi
+ AC_SUBST([NET_IF_H])
+ AM_CONDITIONAL([GL_GENERATE_NET_IF_H], [test -n "$NET_IF_H"])
+])
diff --git a/modules/net_if b/modules/net_if
new file mode 100644
index 0000000..9e30f01
--- /dev/null
+++ b/modules/net_if
@@ -0,0 +1,49 @@
+Description:
+A <net/if.h> for systems lacking it.
+
+Files:
+lib/net_if.in.h
+m4/net_if_h.m4
+
+Depends-on:
+include_next
+sys_socket
+
+configure.ac:
+gl_HEADER_NET_IF
+AC_PROG_MKDIR_P
+
+Makefile.am:
+BUILT_SOURCES += $(NET_IF_H)
+
+# We need the following in order to create <net/if.h> when the system
+# doesn't have one.
+if GL_GENERATE_NET_IF_H
+net/if.h: net_if.in.h $(top_builddir)/config.status
+ $(AM_V_at)$(MKDIR_P) net
+ $(AM_V_GEN)rm -f address@hidden $@ && \
+ { echo '/* DO NOT EDIT! GENERATED AUTOMATICALLY! */'; \
+ sed -e 's|@''GUARD_PREFIX''@|${gl_include_guard_prefix}|g' \
+ -e 's|@''INCLUDE_NEXT''@|$(INCLUDE_NEXT)|g' \
+ -e 's|@''PRAGMA_SYSTEM_HEADER''@|@PRAGMA_SYSTEM_HEADER@|g' \
+ -e 's|@''PRAGMA_COLUMNS''@|@PRAGMA_COLUMNS@|g' \
+ -e 's|@''NEXT_NET_IF_H''@|$(NEXT_NET_IF_H)|g' \
+ -e 's|@''HAVE_NET_IF_H''@|$(HAVE_NET_IF_H)|g' \
+ < $(srcdir)/net_if.in.h; \
+ } > address@hidden && \
+ mv address@hidden $@
+else
+net/if.h: $(top_builddir)/config.status
+ rm -f $@
+endif
+MOSTLYCLEANFILES += net/if.h net/if.h-t
+MOSTLYCLEANDIRS += net
+
+Include:
+<net/if.h>
+
+License:
+LGPLv2+
+
+Maintainer:
+Eric Blake
diff --git a/tests/test-net_if.c b/tests/test-net_if.c
index ee3cdbd..f508642 100644
--- a/tests/test-net_if.c
+++ b/tests/test-net_if.c
@@ -20,11 +20,17 @@
#include <net/if.h>
-#include "signature.h"
+static struct if_nameindex ni;
+
+/* We do not yet have replacements for if_* functions on systems that
+ lack a native <net/if.h>. */
+#if HAVE_NET_IF_H
+# include "signature.h"
SIGNATURE_CHECK (if_freenameindex, void, (struct if_nameindex *));
SIGNATURE_CHECK (if_indextoname, char *, (unsigned int, char *));
SIGNATURE_CHECK (if_nameindex, struct if_nameindex *, (void));
SIGNATURE_CHECK (if_nametoindex, unsigned int, (const char *));
+#endif
#include <stddef.h> /* NULL */
#include <stdio.h> /* fprintf */
@@ -32,6 +38,7 @@ SIGNATURE_CHECK (if_nametoindex, unsigned int, (const char
*));
int
main (int argc, char *argv[])
{
+#if HAVE_NET_IF_H
struct if_nameindex *ifnp, *p;
p = ifnp = if_nameindex ();
@@ -80,6 +87,7 @@ main (int argc, char *argv[])
}
if_freenameindex (ifnp);
+#endif /* HAVE_NET_IF_H */
- return 0;
+ return !IF_NAMESIZE + ni.if_index + !!ni.if_name;
}
--
1.7.11.4