when register to helper, each helper adds port to name. correct form is 'protocol name-port' but irc, sip and tftp adds a iterator value. so it fix it. Signed-off-by: Taehee Yoo <ap420073@xxxxxxxxx> --- net/netfilter/nf_conntrack_irc.c | 2 +- net/netfilter/nf_conntrack_sip.c | 2 +- net/netfilter/nf_conntrack_tftp.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/net/netfilter/nf_conntrack_irc.c b/net/netfilter/nf_conntrack_irc.c index 8b6da27..20ae74f 100644 --- a/net/netfilter/nf_conntrack_irc.c +++ b/net/netfilter/nf_conntrack_irc.c @@ -265,7 +265,7 @@ static int __init nf_conntrack_irc_init(void) if (ports[i] == IRC_PORT) sprintf(irc[i].name, "irc"); else - sprintf(irc[i].name, "irc-%u", i); + sprintf(irc[i].name, "irc-%u", ports[i]); ret = nf_conntrack_helper_register(&irc[i]); if (ret) { diff --git a/net/netfilter/nf_conntrack_sip.c b/net/netfilter/nf_conntrack_sip.c index 3e06402..d523052 100644 --- a/net/netfilter/nf_conntrack_sip.c +++ b/net/netfilter/nf_conntrack_sip.c @@ -1661,7 +1661,7 @@ static int __init nf_conntrack_sip_init(void) if (ports[i] == SIP_PORT) sprintf(sip[i][j].name, "sip"); else - sprintf(sip[i][j].name, "sip-%u", i); + sprintf(sip[i][j].name, "sip-%u", ports[i]); pr_debug("port #%u: %u\n", i, ports[i]); diff --git a/net/netfilter/nf_conntrack_tftp.c b/net/netfilter/nf_conntrack_tftp.c index 36f9640..47239fb 100644 --- a/net/netfilter/nf_conntrack_tftp.c +++ b/net/netfilter/nf_conntrack_tftp.c @@ -136,7 +136,7 @@ static int __init nf_conntrack_tftp_init(void) if (ports[i] == TFTP_PORT) sprintf(tftp[i][j].name, "tftp"); else - sprintf(tftp[i][j].name, "tftp-%u", i); + sprintf(tftp[i][j].name, "tftp-%u", ports[i]); ret = nf_conntrack_helper_register(&tftp[i][j]); if (ret) { -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html