diff options
author | Phil Sutter <phil@nwl.cc> | 2018-09-19 15:16:42 +0200 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2018-09-24 11:23:37 +0200 |
commit | 8c786a351a441ff23ad5d9d1da8cec492f88f542 (patch) | |
tree | ef794dd6b811cba4f842c5a923243cee1f230715 | |
parent | 6fc7762f6f78526e3cb0c189ac2778a6be4c00b5 (diff) |
nfnl_osf: Drop pointless check in xt_osf_strchr()
Although it remains unclear what the original intention behind the
affected code was, but 'tmp + 1' always evaluates true since 'tmp' is a
pointer value.
Cc: Evgeniy Polyakov <johnpol@2ka.mxt.ru>
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Florian Westphal <fw@strlen.de>
-rw-r--r-- | utils/nfnl_osf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/utils/nfnl_osf.c b/utils/nfnl_osf.c index 720e3a38..e65c3201 100644 --- a/utils/nfnl_osf.c +++ b/utils/nfnl_osf.c @@ -141,7 +141,7 @@ static char *xt_osf_strchr(char *ptr, char c) if (tmp) *tmp = '\0'; - while (tmp && tmp + 1 && isspace(*(tmp + 1))) + while (tmp && isspace(*(tmp + 1))) tmp++; return tmp; |