[NETFILTER]: Remove useless comparisons before assignments

Remove unnecessary if() constructs before assignment.

Signed-off-by: Jan Engelhardt <jengelh@gmx.de>
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>

6 files changed, 9 insertions, 21 deletions

diff --git a/net/ipv6/netfilter/ip6t_HL.c b/net/ipv6/netfilter/ip6t_HL.c
index 435750f664dd..7e5d51386f56 100644
--- a/net/ipv6/netfilter/ip6t_HL.c
+++ b/net/ipv6/netfilter/ip6t_HL.c
@@ -52,8 +52,7 @@ static unsigned int ip6t_hl_target(struct sk_buff **pskb,
                         break;
         }
 
-        if (new_hl != ip6h->hop_limit)
-                ip6h->hop_limit = new_hl;
+        ip6h->hop_limit = new_hl;
 
         return IP6T_CONTINUE;
 }
diff --git a/net/netfilter/xt_CLASSIFY.c b/net/netfilter/xt_CLASSIFY.c
index 50de965bb104..195e92990da7 100644
--- a/net/netfilter/xt_CLASSIFY.c
+++ b/net/netfilter/xt_CLASSIFY.c
@@ -33,9 +33,7 @@ target(struct sk_buff **pskb,
 {
         const struct xt_classify_target_info *clinfo = targinfo;
 
-        if ((*pskb)->priority != clinfo->priority)
-                (*pskb)->priority = clinfo->priority;
-
+        (*pskb)->priority = clinfo->priority;
         return XT_CONTINUE;
 }
 
diff --git a/net/netfilter/xt_CONNMARK.c b/net/netfilter/xt_CONNMARK.c
index 0534bfa65cce..795c058b16a5 100644
--- a/net/netfilter/xt_CONNMARK.c
+++ b/net/netfilter/xt_CONNMARK.c
@@ -61,7 +61,7 @@ target(struct sk_buff **pskb,
 #else
                                 nf_conntrack_event_cache(IPCT_MARK, *pskb);
 #endif
-                }
+                        }
                         break;
                 case XT_CONNMARK_SAVE:
                         newmark = (*ctmark & ~markinfo->mask) |
@@ -78,8 +78,7 @@ target(struct sk_buff **pskb,
                 case XT_CONNMARK_RESTORE:
                         mark = (*pskb)->mark;
                         diff = (*ctmark ^ mark) & markinfo->mask;
-                        if (diff != 0)
-                                (*pskb)->mark = mark ^ diff;
+                        (*pskb)->mark = mark ^ diff;
                         break;
                 }
         }
diff --git a/net/netfilter/xt_CONNSECMARK.c b/net/netfilter/xt_CONNSECMARK.c
index a3fe3c334b09..1ab0db641f96 100644
--- a/net/netfilter/xt_CONNSECMARK.c
+++ b/net/netfilter/xt_CONNSECMARK.c
@@ -41,8 +41,7 @@ static void secmark_save(struct sk_buff *skb)
 
                 connsecmark = nf_ct_get_secmark(skb, &ctinfo);
                 if (connsecmark && !*connsecmark)
-                        if (*connsecmark != skb->secmark)
-                                *connsecmark = skb->secmark;
+                        *connsecmark = skb->secmark;
         }
 }
 
@@ -58,8 +57,7 @@ static void secmark_restore(struct sk_buff *skb)
 
                 connsecmark = nf_ct_get_secmark(skb, &ctinfo);
                 if (connsecmark && *connsecmark)
-                        if (skb->secmark != *connsecmark)
-                                skb->secmark = *connsecmark;
+                        skb->secmark = *connsecmark;
         }
 }
 
diff --git a/net/netfilter/xt_MARK.c b/net/netfilter/xt_MARK.c
index 0b48547e8d64..cfc45af357d5 100644
--- a/net/netfilter/xt_MARK.c
+++ b/net/netfilter/xt_MARK.c
@@ -31,9 +31,7 @@ target_v0(struct sk_buff **pskb,
 {
         const struct xt_mark_target_info *markinfo = targinfo;
 
-        if((*pskb)->mark != markinfo->mark)
-                (*pskb)->mark = markinfo->mark;
-
+        (*pskb)->mark = markinfo->mark;
         return XT_CONTINUE;
 }
 
@@ -62,9 +60,7 @@ target_v1(struct sk_buff **pskb,
                 break;
         }
 
-        if((*pskb)->mark != mark)
-                (*pskb)->mark = mark;
-
+        (*pskb)->mark = mark;
         return XT_CONTINUE;
 }
 
diff --git a/net/netfilter/xt_SECMARK.c b/net/netfilter/xt_SECMARK.c
index add752196290..f1131c3a9db5 100644
--- a/net/netfilter/xt_SECMARK.c
+++ b/net/netfilter/xt_SECMARK.c
@@ -47,9 +47,7 @@ static unsigned int target(struct sk_buff **pskb, const struct net_device *in,
                 BUG();
         }
 
-        if ((*pskb)->secmark != secmark)
-                (*pskb)->secmark = secmark;
-
+        (*pskb)->secmark = secmark;
         return XT_CONTINUE;
 }