* [PATCH] ip_tunnel: Fix GCC 8 warning in __ip_tunnel_create()
@ 2018-06-07 0:27 Sultan Alsawaf
0 siblings, 0 replies; only message in thread
From: Sultan Alsawaf @ 2018-06-07 0:27 UTC (permalink / raw)
To: sultanxda; +Cc: netdev, davem, kuznet, yoshfuji
By passing a limit of 2 bytes to strncat, GCC 8 thinks that strncat is
limited to writing fewer bytes than what it's supposed to append to the
name here. However, strncat appends n+1 bytes to the destination, so this
is correct even though it doesn't look right.
Since the bounds are checked on the line above this, just remove the string
bounds checks entirely to silence the warning since they're unneeded.
Signed-off-by: Sultan Alsawaf <sultanxda@gmail.com>
---
net/ipv4/ip_tunnel.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/ipv4/ip_tunnel.c b/net/ipv4/ip_tunnel.c
index 38d906baf1df..c4f5602308ed 100644
--- a/net/ipv4/ip_tunnel.c
+++ b/net/ipv4/ip_tunnel.c
@@ -261,8 +261,8 @@ static struct net_device *__ip_tunnel_create(struct net *net,
} else {
if (strlen(ops->kind) > (IFNAMSIZ - 3))
goto failed;
- strlcpy(name, ops->kind, IFNAMSIZ);
- strncat(name, "%d", 2);
+ strcpy(name, ops->kind);
+ strcat(name, "%d");
}
ASSERT_RTNL();
--
2.17.1
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2018-06-07 0:27 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-07 0:27 [PATCH] ip_tunnel: Fix GCC 8 warning in __ip_tunnel_create() Sultan Alsawaf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).