Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Pablo Neira Ayuso]

On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> compile nfct when we have enabled cttimeout

Strange. The nfnetlink_cttimeout.h file is cached inside the
conntrack-tools tree:

conntrack-tools/include/linux/netfilter/nfnetlink_cttimeout.h

Here this compiles fine with my current tree and using:

./configure --prefix=/usr --disable-cthelper --disable-cttimeout

Please, investigate this issue a bit further.

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Hangbin Liu]

On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > compile nfct when we have enabled cttimeout
> 
> Strange. The nfnetlink_cttimeout.h file is cached inside the
> conntrack-tools tree:
> 
> conntrack-tools/include/linux/netfilter/nfnetlink_cttimeout.h
> 
> Here this compiles fine with my current tree and using:
> 
> ./configure --prefix=/usr --disable-cthelper --disable-cttimeout
> 
> Please, investigate this issue a bit further.

That's weird, on my system without libnetfilter_cttimeout.
# ./configure --prefix=/usr
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes

<snip>

checking for LIBNETFILTER_CONNTRACK... yes
checking for LIBNETFILTER_CTTIMEOUT... no
configure: error: Package requirements (libnetfilter_cttimeout >= 1.0.0) were not met:

No package 'libnetfilter_cttimeout' found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables LIBNETFILTER_CTTIMEOUT_CFLAGS
and LIBNETFILTER_CTTIMEOUT_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

# ./configure --prefix=/usr --disable-cthelper --disable-cttimeout

<snip>

config.status: executing depfiles commands
config.status: executing libtool commands

conntrack-tools configuration:
  userspace conntrack helper support:   no
  conntrack timeout support:            no

# make
Making all in extensions
make[1]: Entering directory `/root/conntrack-tools/extensions'
  CC       libct_proto_tcp.lo
  CCLD     libct_proto_tcp.la

<snip>

  YACC     read_config_yy.c
updating read_config_yy.h
  CC       read_config_yy.o
  LEX      read_config_lex.c
  CC       read_config_lex.o
read_config_lex.c: In function ‘yylex’:
read_config_lex.c:2333:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
    for ( yyl = 0; yyl < yyleng; ++yyl )
                       ^
read_config_lex.c: In function ‘yy_scan_bytes’:
read_config_lex.c:3658:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for ( i = 0; i < _yybytes_len; ++i )
                 ^
  CC       stack.o
  CCLD     conntrackd
  CC       nfct.o
nfct.c:26:59: fatal error: libnetfilter_cttimeout/libnetfilter_cttimeout.h: No such file or directory
 #include <libnetfilter_cttimeout/libnetfilter_cttimeout.h>
                                                           ^
compilation terminated.
make[2]: *** [nfct.o] Error 1
make[2]: Leaving directory `/root/conntrack-tools/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/root/conntrack-tools/src'
make: *** [all-recursive] Error 1

It didn't check our library path, should we add it in LIBRARY_PATH?
-- 

Thanks & Best Regards
Hangbin Liu <liuhangbin@gmail.com>

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Pablo Neira Ayuso]

On Fri, May 09, 2014 at 07:54:11PM +0800, Hangbin Liu wrote:
> On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> > On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > > compile nfct when we have enabled cttimeout
> > 
> > Strange. The nfnetlink_cttimeout.h file is cached inside the
> > conntrack-tools tree:
> > 
> > conntrack-tools/include/linux/netfilter/nfnetlink_cttimeout.h
> > 
> > Here this compiles fine with my current tree and using:
> > 
> > ./configure --prefix=/usr --disable-cthelper --disable-cttimeout
> > 
> > Please, investigate this issue a bit further.
> 
> That's weird, on my system without libnetfilter_cttimeout.
> # ./configure --prefix=/usr
> checking build system type... x86_64-unknown-linux-gnu
> checking host system type... x86_64-unknown-linux-gnu
> checking for a BSD-compatible install... /usr/bin/install -c
> checking whether build environment is sane... yes
> 
> <snip>
> 
> checking for LIBNETFILTER_CONNTRACK... yes
> checking for LIBNETFILTER_CTTIMEOUT... no
> configure: error: Package requirements (libnetfilter_cttimeout >= 1.0.0) were not met:
> 
> No package 'libnetfilter_cttimeout' found
> 
> Consider adjusting the PKG_CONFIG_PATH environment variable if you
> installed software in a non-standard prefix.
> 
> Alternatively, you may set the environment variables LIBNETFILTER_CTTIMEOUT_CFLAGS
> and LIBNETFILTER_CTTIMEOUT_LIBS to avoid the need to call pkg-config.
> See the pkg-config man page for more details.
> 
> # ./configure --prefix=/usr --disable-cthelper --disable-cttimeout
> 
> <snip>
> 
> config.status: executing depfiles commands
> config.status: executing libtool commands
> 
> conntrack-tools configuration:
>   userspace conntrack helper support:   no
>   conntrack timeout support:            no
> 
> # make
> Making all in extensions
> make[1]: Entering directory `/root/conntrack-tools/extensions'
>   CC       libct_proto_tcp.lo
>   CCLD     libct_proto_tcp.la
> 
> <snip>
> 
>   YACC     read_config_yy.c
> updating read_config_yy.h
>   CC       read_config_yy.o
>   LEX      read_config_lex.c
>   CC       read_config_lex.o
> read_config_lex.c: In function ‘yylex’:
> read_config_lex.c:2333:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
>     for ( yyl = 0; yyl < yyleng; ++yyl )
>                        ^
> read_config_lex.c: In function ‘yy_scan_bytes’:
> read_config_lex.c:3658:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
>   for ( i = 0; i < _yybytes_len; ++i )
>                  ^
>   CC       stack.o
>   CCLD     conntrackd
>   CC       nfct.o
> nfct.c:26:59: fatal error: libnetfilter_cttimeout/libnetfilter_cttimeout.h: No such file or directory
>  #include <libnetfilter_cttimeout/libnetfilter_cttimeout.h>
>                                                            ^

OK, I see. That's not nfnetlink_cttimeout.h, it's
libnetfilter_cttimeout.h.

Let me check if I can come up with some alternative solution.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Hangbin Liu]

On Fri, May 09, 2014 at 03:59:42PM +0200, Pablo Neira Ayuso wrote:
> On Fri, May 09, 2014 at 07:54:11PM +0800, Hangbin Liu wrote:
> > On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> > > On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > > > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > > > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > > > compile nfct when we have enabled cttimeout
> 
> OK, I see. That's not nfnetlink_cttimeout.h, it's
> libnetfilter_cttimeout.h.

Ah, sorry, my mistake
> 
> Let me check if I can come up with some alternative solution.

-- 

Thanks & Best Regards
Hangbin Liu <liuhangbin@gmail.com>

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Pablo Neira Ayuso]

On Sat, May 10, 2014 at 10:51:23AM +0800, Hangbin Liu wrote:
> On Fri, May 09, 2014 at 03:59:42PM +0200, Pablo Neira Ayuso wrote:
> > On Fri, May 09, 2014 at 07:54:11PM +0800, Hangbin Liu wrote:
> > > On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> > > > On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > > > > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > > > > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > > > > compile nfct when we have enabled cttimeout
> > 
> > OK, I see. That's not nfnetlink_cttimeout.h, it's
> > libnetfilter_cttimeout.h.
> 
> Ah, sorry, my mistake

Please, give a try to the attached patch.

Thanks.

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Pablo Neira Ayuso]

[-- Attachment #1: Type: text/plain, Size: 861 bytes --]

On Mon, May 12, 2014 at 05:23:54PM +0200, Pablo Neira Ayuso wrote:
> On Sat, May 10, 2014 at 10:51:23AM +0800, Hangbin Liu wrote:
> > On Fri, May 09, 2014 at 03:59:42PM +0200, Pablo Neira Ayuso wrote:
> > > On Fri, May 09, 2014 at 07:54:11PM +0800, Hangbin Liu wrote:
> > > > On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> > > > > On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > > > > > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > > > > > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > > > > > compile nfct when we have enabled cttimeout
> > > 
> > > OK, I see. That's not nfnetlink_cttimeout.h, it's
> > > libnetfilter_cttimeout.h.
> > 
> > Ah, sorry, my mistake
> 
> Please, give a try to the attached patch.

Now, really attaching the patch.

[-- Attachment #2: 0001-nfct-remove-unneeded-included-header.patch --]
[-- Type: text/x-diff, Size: 796 bytes --]

>From 34a8e200eee54b4bbecadf52bba8901cae129795 Mon Sep 17 00:00:00 2001
From: Pablo Neira Ayuso <pablo@netfilter.org>
Date: Mon, 12 May 2014 17:21:21 +0200
Subject: [PATCH] nfct: remove unneeded included header

This fixes a compilation breakage when libnetfilter_cttimeout.h is
not installed.

Reported-by: Hangbin Liu <liuhangbin@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
---
 src/nfct.c |    2 --
 1 file changed, 2 deletions(-)

diff --git a/src/nfct.c b/src/nfct.c
index 19e44be..533d75d 100644
--- a/src/nfct.c
+++ b/src/nfct.c
@@ -22,8 +22,6 @@
 #include <errno.h>
 
 #include <libmnl/libmnl.h>
-#include <linux/netfilter/nfnetlink_cttimeout.h>
-#include <libnetfilter_cttimeout/libnetfilter_cttimeout.h>
 
 #include "linux_list.h"
 #include "nfct.h"
-- 
1.7.10.4

Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout

[Hangbin Liu]

On Mon, May 12, 2014 at 05:24:21PM +0200, Pablo Neira Ayuso wrote:
> On Mon, May 12, 2014 at 05:23:54PM +0200, Pablo Neira Ayuso wrote:
> > On Sat, May 10, 2014 at 10:51:23AM +0800, Hangbin Liu wrote:
> > > On Fri, May 09, 2014 at 03:59:42PM +0200, Pablo Neira Ayuso wrote:
> > > > On Fri, May 09, 2014 at 07:54:11PM +0800, Hangbin Liu wrote:
> > > > > On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> > > > > > On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > > > > > > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > > > > > > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > > > > > > compile nfct when we have enabled cttimeout
> > > > 
> > > > OK, I see. That's not nfnetlink_cttimeout.h, it's
> > > > libnetfilter_cttimeout.h.
> > > 
> > > Ah, sorry, my mistake
> > 
> > Please, give a try to the attached patch.
> 
> Now, really attaching the patch.

Good, this works for me.

> From 34a8e200eee54b4bbecadf52bba8901cae129795 Mon Sep 17 00:00:00 2001
> From: Pablo Neira Ayuso <pablo@netfilter.org>
> Date: Mon, 12 May 2014 17:21:21 +0200
> Subject: [PATCH] nfct: remove unneeded included header
> 
> This fixes a compilation breakage when libnetfilter_cttimeout.h is
> not installed.
> 
> Reported-by: Hangbin Liu <liuhangbin@gmail.com>
> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
> ---
>  src/nfct.c |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/src/nfct.c b/src/nfct.c
> index 19e44be..533d75d 100644
> --- a/src/nfct.c
> +++ b/src/nfct.c
> @@ -22,8 +22,6 @@
>  #include <errno.h>
>  
>  #include <libmnl/libmnl.h>
> -#include <linux/netfilter/nfnetlink_cttimeout.h>
> -#include <libnetfilter_cttimeout/libnetfilter_cttimeout.h>
>  
>  #include "linux_list.h"
>  #include "nfct.h"
> -- 
> 1.7.10.4
> 


-- 

Thanks & Best Regards
Hangbin Liu <liuhangbin@gmail.com>