Commit 76a2d3bc authored by Changli Gao's avatar Changli Gao Committed by Patrick McHardy

netfilter: nf_nat: don't use atomic bit operation

As we own the conntrack and the others can't see it until we confirm it,
we don't need to use atomic bit operation on ct->status.
Signed-off-by: default avatarChangli Gao <xiaosuo@gmail.com>
Signed-off-by: default avatarPatrick McHardy <kaber@trash.net>
parent 0f8e8004
...@@ -21,9 +21,9 @@ static inline int nf_nat_initialized(struct nf_conn *ct, ...@@ -21,9 +21,9 @@ static inline int nf_nat_initialized(struct nf_conn *ct,
enum nf_nat_manip_type manip) enum nf_nat_manip_type manip)
{ {
if (manip == IP_NAT_MANIP_SRC) if (manip == IP_NAT_MANIP_SRC)
return test_bit(IPS_SRC_NAT_DONE_BIT, &ct->status); return ct->status & IPS_SRC_NAT_DONE_BIT;
else else
return test_bit(IPS_DST_NAT_DONE_BIT, &ct->status); return ct->status & IPS_DST_NAT_DONE_BIT;
} }
struct nlattr; struct nlattr;
......
...@@ -323,9 +323,9 @@ nf_nat_setup_info(struct nf_conn *ct, ...@@ -323,9 +323,9 @@ nf_nat_setup_info(struct nf_conn *ct,
/* It's done. */ /* It's done. */
if (maniptype == IP_NAT_MANIP_DST) if (maniptype == IP_NAT_MANIP_DST)
set_bit(IPS_DST_NAT_DONE_BIT, &ct->status); ct->status |= IPS_DST_NAT_DONE_BIT;
else else
set_bit(IPS_SRC_NAT_DONE_BIT, &ct->status); ct->status |= IPS_SRC_NAT_DONE_BIT;
return NF_ACCEPT; return NF_ACCEPT;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment