Use strcpy() rather the sprintf() for the case where name is getting generated. Fix indentation. Signed-off-by: Stephen Hemminger --- a/net/ipv4/ip_gre.c 2010-11-28 11:28:48.684091117 -0800 +++ b/net/ipv4/ip_gre.c 2010-11-28 11:29:21.839822531 -0800 @@ -405,11 +405,11 @@ static struct ip_tunnel *ipgre_tunnel_lo if (parms->name[0]) strlcpy(name, parms->name, IFNAMSIZ); else - sprintf(name, "gre%%d"); + strcpy(name, "gre%d"); dev = alloc_netdev(sizeof(*t), name, ipgre_tunnel_setup); if (!dev) - return NULL; + return NULL; dev_net_set(dev, net); -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html