Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fc3893fd authored by Florian Westphal's avatar Florian Westphal Committed by Pablo Neira Ayuso
Browse files

netfilter: conntrack: remove empty pernet fini stubs 



after moving sysctl handling into single place, the init functions
can't fail anymore and some of the fini functions are empty.

Remove them and change return type to void.
This also simplifies error unwinding in conntrack module init path.

Signed-off-by: default avatarFlorian Westphal <fw@strlen.de>
Signed-off-by: default avatarPablo Neira Ayuso <pablo@netfilter.org>
parent cb2833ed

include/net/netfilter/nf_conntrack_acct.h

+1 −2
Original line number Diff line number Diff line
@@ -58,8 +58,7 @@ static inline void nf_ct_set_acct(struct net *net, bool enable) 
	net->ct.sysctl_acct = enable;

}



int nf_conntrack_acct_pernet_init(struct net *net);

void nf_conntrack_acct_pernet_fini(struct net *net);

void nf_conntrack_acct_pernet_init(struct net *net);



int nf_conntrack_acct_init(void);

void nf_conntrack_acct_fini(void);

include/net/netfilter/nf_conntrack_ecache.h

+2 −5
Original line number Diff line number Diff line
@@ -142,7 +142,7 @@ void nf_ct_expect_event_report(enum ip_conntrack_expect_events event, 
			       struct nf_conntrack_expect *exp,

			       u32 portid, int report);



int nf_conntrack_ecache_pernet_init(struct net *net);

void nf_conntrack_ecache_pernet_init(struct net *net);

void nf_conntrack_ecache_pernet_fini(struct net *net);



int nf_conntrack_ecache_init(void);
@@ -182,10 +182,7 @@ static inline void nf_ct_expect_event_report(enum ip_conntrack_expect_events e, 
 					     u32 portid,

 					     int report) {}



static inline int nf_conntrack_ecache_pernet_init(struct net *net)

{

	return 0;

}

static inline void nf_conntrack_ecache_pernet_init(struct net *net) {}



static inline void nf_conntrack_ecache_pernet_fini(struct net *net)

{

include/net/netfilter/nf_conntrack_helper.h

+1 −2
Original line number Diff line number Diff line
@@ -124,8 +124,7 @@ static inline void *nfct_help_data(const struct nf_conn *ct) 
	return (void *)help->data;

}



int nf_conntrack_helper_pernet_init(struct net *net);

void nf_conntrack_helper_pernet_fini(struct net *net);

void nf_conntrack_helper_pernet_init(struct net *net);



int nf_conntrack_helper_init(void);

void nf_conntrack_helper_fini(void);

include/net/netfilter/nf_conntrack_timestamp.h

+2 −11
Original line number Diff line number Diff line
@@ -49,21 +49,12 @@ static inline void nf_ct_set_tstamp(struct net *net, bool enable) 
}



#ifdef CONFIG_NF_CONNTRACK_TIMESTAMP

int nf_conntrack_tstamp_pernet_init(struct net *net);

void nf_conntrack_tstamp_pernet_fini(struct net *net);

void nf_conntrack_tstamp_pernet_init(struct net *net);



int nf_conntrack_tstamp_init(void);

void nf_conntrack_tstamp_fini(void);

#else

static inline int nf_conntrack_tstamp_pernet_init(struct net *net)

{

	return 0;

}



static inline void nf_conntrack_tstamp_pernet_fini(struct net *net)

{

	return;

}

static inline void nf_conntrack_tstamp_pernet_init(struct net *net) {}



static inline int nf_conntrack_tstamp_init(void)

{

net/netfilter/nf_conntrack_acct.c

+1 −6
Original line number Diff line number Diff line
@@ -31,14 +31,9 @@ static const struct nf_ct_ext_type acct_extend = { 
	.id	= NF_CT_EXT_ACCT,

};



int nf_conntrack_acct_pernet_init(struct net *net)

void nf_conntrack_acct_pernet_init(struct net *net)

{

	net->ct.sysctl_acct = nf_ct_acct;

	return 0;

}



void nf_conntrack_acct_pernet_fini(struct net *net)

{

}



int nf_conntrack_acct_init(void)