checking if 'name' parameter was provided with 'setoption' before pro…

…ceeding with execution adressing issue LeelaChessZero#1843
KarlKfoury · Aug 31, 2024 · ecf3edd · ecf3edd
1 parent d298c04
commit ecf3edd
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/chess/uciloop.cc b/src/chess/uciloop.cc
@@ -153,8 +153,13 @@ bool UciLoop::DispatchCommand(
   } else if (command == "isready") {
     CmdIsReady();
   } else if (command == "setoption") {
+if (GetOrEmpty(params, "name").empty()) {
+      throw Exception("setoption requires name");
+    }
+    else {
     CmdSetOption(GetOrEmpty(params, "name"), GetOrEmpty(params, "value"),
                  GetOrEmpty(params, "context"));
+  } 
   } else if (command == "ucinewgame") {
     CmdUciNewGame();
   } else if (command == "position") {