Added port validation

cmd/minikube/cmd/start.go

@@ -1118,6 +1118,10 @@ func validateFlags(cmd *cobra.Command, drvName string) {
 		viper.Set(imageRepository, validateImageRepository(viper.GetString(imageRepository)))
 	}
 
+	if cmd.Flags().Changed(ports) {
+		validatePorts(viper.GetStringSlice(ports))
+	}
+
 	if cmd.Flags().Changed(containerRuntime) {
 		runtime := strings.ToLower(viper.GetString(containerRuntime))
 
@@ -1314,6 +1318,24 @@ func validateListenAddress(listenAddr string) {
 	}
 }
 
+// This function validates that the --ports are not below 1024 for the host and not outside range
+func validatePorts(ports []string) {
+	for _, portDuplet := range ports {
+		for i, port := range strings.Split(portDuplet, ":") {
+			p, err := strconv.Atoi(port)
+			if err != nil {
+				exit.Message(reason.Usage, "Sorry, one of the ports provided with --ports flag is not valid {{.ports}}", out.V{"ports": ports})
+			}
+			if p > 65535 {
+				exit.Message(reason.Usage, "Sorry, one of the ports provided with --ports flag is outside range {{.ports}}", out.V{"ports": ports})
+			}
+			if p < 1024 && i == 0 {
+				exit.Message(reason.Usage, "Sorry, you cannot use privileged ports on the host (below 1024) {{.ports}}", out.V{"ports": ports})
+			}
+		}
+	}
+}
+
 // This function validates that the --insecure-registry follows one of the following formats:
 // "<ip>[:<port>]" "<hostname>[:<port>]" "<network>/<netmask>"
 func validateInsecureRegistry() {