Update NetworkChangeName to be more similar to my original (moving IFNAMSIZ constant outside the function like it should've been)

Docker-DCO-1.1-Signed-off-by: Andrew Page <admwiggin@gmail.com> (github: tianon)
Upstream-commit: 3a7c144e9992591f32daf2d4f1b35b7b6520a07e
Component: engine

components/engine/pkg/netlink/netlink_linux.go

@@ -662,15 +662,17 @@ func getIfSocket() (int, error) {
 	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)
-	IFNAMSIZ := 16
 
-	data := [32]byte{}
+	data := [IFNAMSIZ * 2]byte{}
 	copy(data[:IFNAMSIZ-1], oldName)
 	copy(data[IFNAMSIZ:IFNAMSIZ*2-1], newName)