diff mbox

[RFC,netfilter] Fix Unknown symbols in nf_reject_ipvX modules

Message ID 1414135732.19476.10.camel@TP420
State Not Applicable
Delegated to: Pablo Neira
Headers show

Commit Message

Li Zhong Oct. 24, 2014, 7:28 a.m. UTC
Commit c8d7b98bec43 moved nf_send_resetX() to modules without module
license, which causes following errors seen on my system: 

[   26.926014] nf_reject_ipv4: Unknown symbol rcu_read_lock_bh_held (err 0)
[   26.926022] nf_reject_ipv4: Unknown symbol rcu_read_lock_held (err 0)
[   26.926030] nf_reject_ipv4: Unknown symbol ip_local_out_sk (err 0)

The code below adds MODULE_LICENSE("GPL") to fix this issue. 

Signed-off-by: Li Zhong <zhong@linux.vnet.ibm.com>
---
 net/ipv4/netfilter/nf_reject_ipv4.c | 3 +++
 net/ipv6/netfilter/nf_reject_ipv6.c | 3 +++
 2 files changed, 6 insertions(+)




--
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

Comments

Pablo Neira Ayuso Oct. 24, 2014, 9:47 a.m. UTC | #1
On Fri, Oct 24, 2014 at 03:28:52PM +0800, Li Zhong wrote:
> Commit c8d7b98bec43 moved nf_send_resetX() to modules without module
> license, which causes following errors seen on my system: 
> 
> [   26.926014] nf_reject_ipv4: Unknown symbol rcu_read_lock_bh_held (err 0)
> [   26.926022] nf_reject_ipv4: Unknown symbol rcu_read_lock_held (err 0)
> [   26.926030] nf_reject_ipv4: Unknown symbol ip_local_out_sk (err 0)
> 
> The code below adds MODULE_LICENSE("GPL") to fix this issue. 

We already sent a similar patch that should hit Linus' tree soon.

Sorry for the inconvenience.
--
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
Li Zhong Oct. 27, 2014, 4:50 a.m. UTC | #2
On δΊ”, 2014-10-24 at 11:47 +0200, Pablo Neira Ayuso wrote:
> On Fri, Oct 24, 2014 at 03:28:52PM +0800, Li Zhong wrote:
> > Commit c8d7b98bec43 moved nf_send_resetX() to modules without module
> > license, which causes following errors seen on my system: 
> > 
> > [   26.926014] nf_reject_ipv4: Unknown symbol rcu_read_lock_bh_held (err 0)
> > [   26.926022] nf_reject_ipv4: Unknown symbol rcu_read_lock_held (err 0)
> > [   26.926030] nf_reject_ipv4: Unknown symbol ip_local_out_sk (err 0)
> > 
> > The code below adds MODULE_LICENSE("GPL") to fix this issue. 
> 
> We already sent a similar patch that should hit Linus' tree soon.

Ah, Ok, good to know. 

Sorry for the noise :)

> 
> Sorry for the inconvenience.
> 


--
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
diff mbox

Patch

diff --git a/net/ipv4/netfilter/nf_reject_ipv4.c b/net/ipv4/netfilter/nf_reject_ipv4.c
index b023b4e..d5df20d 100644
--- a/net/ipv4/netfilter/nf_reject_ipv4.c
+++ b/net/ipv4/netfilter/nf_reject_ipv4.c
@@ -11,6 +11,9 @@ 
 #include <net/route.h>
 #include <net/dst.h>
 #include <linux/netfilter_ipv4.h>
+#include <linux/module.h>
+
+MODULE_LICENSE("GPL");
 
 /* Send RST reply */
 void nf_send_reset(struct sk_buff *oldskb, int hook)
diff --git a/net/ipv6/netfilter/nf_reject_ipv6.c b/net/ipv6/netfilter/nf_reject_ipv6.c
index 5f5f043..e3e5683 100644
--- a/net/ipv6/netfilter/nf_reject_ipv6.c
+++ b/net/ipv6/netfilter/nf_reject_ipv6.c
@@ -10,6 +10,9 @@ 
 #include <net/ip6_fib.h>
 #include <net/ip6_checksum.h>
 #include <linux/netfilter_ipv6.h>
+#include <linux/module.h>
+
+MODULE_LICENSE("GPL");
 
 void nf_send_reset6(struct net *net, struct sk_buff *oldskb, int hook)
 {