Nope, we need the explicit null terminator.
On 10 Feb 2014 12:17, "Tibor Vass" <notifications@github.com> wrote:

> In pkg/netlink/netlink_linux.go:
>
> > +
> > +	return -1, sErr
> > +}
> > +
> > +// from <net/if.h>
> > +const IFNAMSIZ = 16
> > +
> > +func NetworkChangeName(oldName, newName string) error {
> > +	fd, err := getIfSocket()
> > +	if err != nil {
> > +		return err
> > +	}
> > +	defer syscall.Close(fd)
> > +
> > +	data := [IFNAMSIZ * 2]byte{}
> > +	copy(data[:IFNAMSIZ-1], oldName)
>
> Shouldn't this be s/IFNAMSIZ-1/IFNAMSIZ ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dotcloud/docker/pull/4011/files#r9597500>
> .
>