Return a better error message is unix socket path is too long.

mlaventure · mlaventure · commit 3c3a67649097 · 2018-03-30T09:00:02.000-07:00
Signed-off-by: Kenfe-Mickael Laventure &lt;mickael.laventure@gmail.com&gt;
diff --git a/cmd/containerd-shim/main_unix.go b/cmd/containerd-shim/main_unix.go
@@ -162,6 +162,9 @@ func serve(server *ttrpc.Server, path string) error {
 		l, err = net.FileListener(os.NewFile(3, "socket"))
 		path = "[inherited from parent]"
 	} else {
+		if len(path) > 106 {
+			return errors.Errorf("%q: unix socket path too long (> 106)", path)
+		}
 		l, err = net.Listen("unix", "\x00"+path)
 	}
 	if err != nil {
diff --git a/linux/shim/client/client.go b/linux/shim/client/client.go
@@ -145,7 +145,7 @@ func newCommand(binary, daemonAddress string, debug bool, config shim.Config, so
 
 func newSocket(address string) (*net.UnixListener, error) {
 	if len(address) > 106 {
-		return nil, errors.Errorf("%q: unix socket path too long (limit 106)", address)
+		return nil, errors.Errorf("%q: unix socket path too long (> 106)", address)
 	}
 	l, err := net.Listen("unix", "\x00"+address)
 	if err != nil {
diff --git a/sys/socket_unix.go b/sys/socket_unix.go
@@ -23,11 +23,16 @@ import (
 	"os"
 	"path/filepath"
 
+	"github.com/pkg/errors"
 	"golang.org/x/sys/unix"
 )
 
 // CreateUnixSocket creates a unix socket and returns the listener
 func CreateUnixSocket(path string) (net.Listener, error) {
+	// BSDs have a 104 limit
+	if len(path) > 104 {
+		return nil, errors.Errorf("%q: unix socket path too long (> 106)", path)
+	}
 	if err := os.MkdirAll(filepath.Dir(path), 0660); err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ func newCommand(binary, daemonAddress string, debug bool, config shim.Config, so`
`145`	`145`
`146`	`146`	`func newSocket(address string) (*net.UnixListener, error) {`
`147`	`147`	`if len(address) > 106 {`
`148`		`- return nil, errors.Errorf("%q: unix socket path too long (limit 106)", address)`
	`148`	`+ return nil, errors.Errorf("%q: unix socket path too long (> 106)", address)`
`149`	`149`	`}`
`150`	`150`	`l, err := net.Listen("unix", "\x00"+address)`
`151`	`151`	`if err != nil {`