added return in startMinecraftServer if error

small bug correction this needs to be improved/expanded #45
gekware · Oct 14, 2020 · d17ce4e · d17ce4e
1 parent a2b668c
commit d17ce4e
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/go-version/minecraft-server-hibernation.go b/go-version/minecraft-server-hibernation.go
@@ -81,10 +81,15 @@ func startMinecraftServer() {
 		err := exec.Command("/bin/bash", "-c", config.Tomodify.StartMinecraftServerLin).Run()
 		if err != nil {
 			log.Printf("stopEmptyMinecraftServer: error starting minecraft server: %v\n", err)
+			return
 		}
 	} else if runtime.GOOS == "windows" {
 		cmd := exec.Command(strings.Split(config.Tomodify.StartMinecraftServerWin, " ")[0], strings.Split(config.Tomodify.StartMinecraftServerWin, " ")[1:]...)
-		cmdIn, _ = cmd.StdinPipe()
+		cmdIn, err := cmd.StdinPipe()
+		if err != nil {
+			log.Printf("stopEmptyMinecraftServer: error creating StdinPipe: %v\n", err)
+			return
+		}
 		cmd.Start()
 	} else {
 		log.Print("stopEmptyMinecraftServer: error: OS not supported!")