Linux Netfilter discussions
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Thomas Berg <thomas.berg@branndal.se>
Cc: /dev/rob0 <rob0@gmx.co.uk>, netfilter@vger.kernel.org
Subject: Re: ULOG The kernel doesn't support a certain ebtables extension, consider recompiling your kernel or insmod the extension.
Date: Tue, 01 Sep 2009 14:35:41 +0200	[thread overview]
Message-ID: <4A9D151D.30808@trash.net> (raw)
In-Reply-To: <4A9D10DF.1050307@branndal.se>

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

Thomas Berg wrote:
> Hi
> 
> trying this
> 
> rmmod ipt_ulog
> 
> modprobe ebt_ulog
> 
> ebtables 14108 3 ebtable_broute,ebtable_nat,ebtable_filter
> x_tables 14108 5 ebt_ulog,ebt_nflog,ebt_ip,ebt_log,ebtables
> 
> 
> module is then loaded, great but still the same error when trying to add
> a ebtables rule with ulog.
> 
> br0:~# /sbin/ebtables --modprobe /sbin/modprobe -A FORWARD
> --in-interface eth1.4 --protocol IPv4 --ip-protocol udp
> --ip-destination-port 67 -j DROP --ulog
> The kernel doesn't support a certain ebtables extension, consider
> recompiling your kernel or insmod the extension.

This patch should fix it.



[-- Attachment #2: x --]
[-- Type: text/plain, Size: 873 bytes --]

commit 8a56df0ae1690f8f42a3c6c4532f4b06f93febea
Author: Patrick McHardy <kaber@trash.net>
Date:   Tue Sep 1 14:34:01 2009 +0200

    netfilter: ebt_ulog: fix checkentry return value
    
    Commit 19eda87 (netfilter: change return types of check functions for
    Ebtables extensions) broke the ebtables ulog module by missing a return
    value conversion.
    
    Signed-off-by: Patrick McHardy <kaber@trash.net>

diff --git a/net/bridge/netfilter/ebt_ulog.c b/net/bridge/netfilter/ebt_ulog.c
index 133eeae..ce50688 100644
--- a/net/bridge/netfilter/ebt_ulog.c
+++ b/net/bridge/netfilter/ebt_ulog.c
@@ -266,7 +266,7 @@ static bool ebt_ulog_tg_check(const struct xt_tgchk_param *par)
 	if (uloginfo->qthreshold > EBT_ULOG_MAX_QLEN)
 		uloginfo->qthreshold = EBT_ULOG_MAX_QLEN;
 
-	return 0;
+	return true;
 }
 
 static struct xt_target ebt_ulog_tg_reg __read_mostly = {

  reply	other threads:[~2009-09-01 12:35 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-31 20:15 ULOG The kernel doesn't support a certain ebtables extension, consider recompiling your kernel or insmod the extension Thomas Berg
2009-08-31 20:34 ` /dev/rob0
2009-09-01  7:47   ` Thomas Berg
2009-09-01 11:43     ` Patrick McHardy
2009-09-01 12:17       ` Thomas Berg
2009-09-01 12:35         ` Patrick McHardy [this message]
2009-09-01 13:20           ` Thomas Berg
2009-09-01 13:21             ` Patrick McHardy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A9D151D.30808@trash.net \
    --to=kaber@trash.net \
    --cc=netfilter@vger.kernel.org \
    --cc=rob0@gmx.co.uk \
    --cc=thomas.berg@branndal.se \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox