ContainerTop: disallow --no-headers

[v2: add --no-header, found by accident]
[v3: add --no-heading, found in ps man page]
[v4: add other 5 forms, found in procps sources]

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

daemon/top_unix.go

@@ -285,6 +285,22 @@ func (daemon *Daemon) ContainerTop(name string, psArgs string) (*container.Conta
 
 	args := strings.Split(psArgs, " ")
 
+	// can't work without headers
+	for _, arg := range args {
+		switch arg {
+		case
+			"--no-header",
+			"--no-headers",
+			"--no-heading",
+			"--no-headings",
+			"--noheader",
+			"--noheaders",
+			"--noheading",
+			"--noheadings":
+			return nil, errdefs.System(errors.New("option " + arg + " is not allowed"))
+		}
+	}
+
 	var extraPidColumn bool
 	if customArgs && customFields(args) {
 		// make sure the PID field is shown in the first column

integration/container/top_test.go

@@ -110,6 +110,10 @@ func TestContainerTop(t *testing.T) {
 			opts:      []string{"-o", "sasha"},
 			expectErr: true,
 		},
+		{
+			opts:      []string{"--no-headers"},
+			expectErr: true,
+		},
 	}
 
 	for _, c := range cases {