diff options
| author |   Pravin B Shelar <pshelar@nicira.com> | 2013-03-28 08:21:46 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2013-03-29 15:24:28 -0400 |
| commit | 54a5d3828995c0df4f8e12a5d91b1c42f0f323d6 (patch) | |
| tree | 40d57f9ab4a605ef62ff6382cf35a3315cf67599 /net | |
| parent | net: simplify the getting percpu of flow_cache (diff) | |
| download | linux-dev-54a5d3828995c0df4f8e12a5d91b1c42f0f323d6.tar.xz linux-dev-54a5d3828995c0df4f8e12a5d91b1c42f0f323d6.zip | |
ip_tunnel: Fix off-by-one error in forming dev name.
As Ben pointed out following patch fixes bug in checking device
name length limits while forming tunnel device name.
CC: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/ipv4/ip_tunnel.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/ip_tunnel.c b/net/ipv4/ip_tunnel.c index 9d96b6853f21..e4147ec1665a 100644 --- a/net/ipv4/ip_tunnel.c +++ b/net/ipv4/ip_tunnel.c @@ -284,7 +284,7 @@ static struct net_device *__ip_tunnel_create(struct net *net, if (parms->name[0]) strlcpy(name, parms->name, IFNAMSIZ); else { - if (strlen(ops->kind) + 3 >= IFNAMSIZ) { + if (strlen(ops->kind) > (IFNAMSIZ - 3)) { err = -E2BIG; goto failed; } |