diff options
| author |   Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2016-02-23 13:40:10 +0530 |
|---|---|---|
| committer |   Pablo Neira Ayuso <pablo@netfilter.org> | 2016-02-29 13:59:43 +0100 |
| commit | 8ee225e78531981e479e79b74354b63e188b4ee2 (patch) | |
| tree | c5a24c8f105c2b1a76f47172038ce1e14824f1bb /net/netfilter/xt_osf.c | |
| parent | netfilter: meta: add PRANDOM support (diff) | |
| download | linux-dev-8ee225e78531981e479e79b74354b63e188b4ee2.tar.xz linux-dev-8ee225e78531981e479e79b74354b63e188b4ee2.zip | |
netfilter: xt_osf: remove unused variable
While building with W=1 we got the warning:
net/netfilter/xt_osf.c:265:9: warning: variable 'loop_cont' set but not used
The local variable loop_cont was only initialized and then assigned a
value but was never used or checked after that.
While removing the variable, the case of OSFOPT_TS was not removed so
that it will serve as a reminder to us that we can do something in that
particular case.
Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to '')
| -rw-r--r-- | net/netfilter/xt_osf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/netfilter/xt_osf.c b/net/netfilter/xt_osf.c index 4e3c3affd285..2455b69b5810 100644 --- a/net/netfilter/xt_osf.c +++ b/net/netfilter/xt_osf.c @@ -262,7 +262,6 @@ xt_osf_match_packet(const struct sk_buff *skb, struct xt_action_param *p) if (f->opt[optnum].kind == (*optp)) { __u32 len = f->opt[optnum].length; const __u8 *optend = optp + len; - int loop_cont = 0; fmatch = FMATCH_OK; @@ -275,7 +274,6 @@ xt_osf_match_packet(const struct sk_buff *skb, struct xt_action_param *p) mss = ntohs((__force __be16)mss); break; case OSFOPT_TS: - loop_cont = 1; break; } |