summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King2019-04-09 14:59:12 +0200
committerDavid S. Miller2019-04-11 23:01:08 +0200
commit62720b12d20aecebc2e74642c37a3dc84717ac7a (patch)
tree9ec15b15145412a4bc1f2f80fc32be2ab73db50d
parenttipc: use standard write_lock & unlock functions when creating node (diff)
downloadkernel-qcow2-linux-62720b12d20aecebc2e74642c37a3dc84717ac7a.tar.gz
kernel-qcow2-linux-62720b12d20aecebc2e74642c37a3dc84717ac7a.tar.xz
kernel-qcow2-linux-62720b12d20aecebc2e74642c37a3dc84717ac7a.zip
dns: remove redundant zero length namelen check
The zero namelen check is redundant as it has already been checked for zero at the start of the function. Remove the redundant check. Addresses-Coverity: ("Logically Dead Code") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/dns_resolver/dns_query.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/net/dns_resolver/dns_query.c b/net/dns_resolver/dns_query.c
index 76338c38738a..19aa32fc1802 100644
--- a/net/dns_resolver/dns_query.c
+++ b/net/dns_resolver/dns_query.c
@@ -94,8 +94,6 @@ int dns_query(const char *type, const char *name, size_t namelen,
desclen += typelen + 1;
}
- if (!namelen)
- namelen = strnlen(name, 256);
if (namelen < 3 || namelen > 255)
return -EINVAL;
desclen += namelen + 1;