Add error check in kubectl proxy on server setup

If "NewServer" returns an error, it will result in a nil pointer
dereference segfault.

A simple way to test the behavior is to prefix the server url with a
colon, ":".

Author: odinuge

hack/.staticcheck_failures

@@ -103,7 +103,6 @@ vendor/k8s.io/kubectl/pkg/cmd/config
 vendor/k8s.io/kubectl/pkg/cmd/edit
 vendor/k8s.io/kubectl/pkg/cmd/exec
 vendor/k8s.io/kubectl/pkg/cmd/get
-vendor/k8s.io/kubectl/pkg/cmd/proxy
 vendor/k8s.io/kubectl/pkg/cmd/rollingupdate
 vendor/k8s.io/kubectl/pkg/cmd/scale
 vendor/k8s.io/kubectl/pkg/cmd/set

staging/src/k8s.io/kubectl/pkg/cmd/proxy/proxy.go

@@ -200,6 +200,10 @@ func (o ProxyOptions) Validate() error {
 func (o ProxyOptions) RunProxy() error {
 	server, err := proxy.NewServer(o.staticDir, o.apiPrefix, o.staticPrefix, o.filter, o.clientConfig, o.keepalive)
 
+	if err != nil {
+		return err
+	}
+
 	// Separate listening from serving so we can report the bound port
 	// when it is chosen by os (eg: port == 0)
 	var l net.Listener
@@ -209,9 +213,8 @@ func (o ProxyOptions) RunProxy() error {
 		l, err = server.ListenUnix(o.unixSocket)
 	}
 	if err != nil {
-		klog.Fatal(err)
+		return err
 	}
 	fmt.Fprintf(o.IOStreams.Out, "Starting to serve on %s\n", l.Addr().String())
-	klog.Fatal(server.ServeOnListener(l))
-	return nil
+	return server.ServeOnListener(l)
 }