[PATCH] Remove inclusion of ip_compat.h for FreeBSD

Dimitry Andric dimitry at andric.com
Sat Feb 7 13:58:48 PST 2015


Hi kcc, kutuzov.viktor.84,

In sanitizer_platform_limits_posix.cc, for FreeBSD the
<netinet/ip_compat.h> header is included unconditionally.  This does not
always work, as the header is not always installed, because it is only
used internally by ipfilter.

None of the code in sanitizer_platform_limits_posix.cc seems to use any
of the definitions in ip_compat.h, so can we please remove the include?

See also: https://svnweb.freebsd.org/changeset/base/277201

http://reviews.llvm.org/D7486

Files:
  lib/sanitizer_common/sanitizer_platform_limits_posix.cc

Index: lib/sanitizer_common/sanitizer_platform_limits_posix.cc
===================================================================
--- lib/sanitizer_common/sanitizer_platform_limits_posix.cc
+++ lib/sanitizer_common/sanitizer_platform_limits_posix.cc
@@ -97,7 +97,6 @@
 # include <sys/link_elf.h>
 # include <netinet/ip_mroute.h>
 # include <netinet/in.h>
-# include <netinet/ip_compat.h>
 # include <net/ethernet.h>
 # include <net/ppp_defs.h>
 # include <glob.h>

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7486.19539.patch
Type: text/x-patch
Size: 465 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150207/d7574923/attachment.bin>


More information about the cfe-commits mailing list