Add comment clarifying null termination

Docker-DCO-1.1-Signed-off-by: Andrew Page <admwiggin@gmail.com> (github: tianon)
creack · Feb 11, 2014 · c626349 · c626349
1 parent f9cd1be
commit c626349
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/pkg/netlink/netlink_linux.go b/pkg/netlink/netlink_linux.go
@@ -731,6 +731,8 @@ func NetworkChangeName(iface *net.Interface, newName string) error {
 	defer syscall.Close(fd)
 
 	data := [IFNAMSIZ * 2]byte{}
+	// the "-1"s here are very important for ensuring we get proper null
+	// termination of our new C strings
 	copy(data[:IFNAMSIZ-1], iface.Name)
 	copy(data[IFNAMSIZ:IFNAMSIZ*2-1], newName)