Make ClientInterface::statenames consistent with the state enum again

Fixes minetest.get_player_information segfault due
to out of bounds access problems, when compiled as debug build.

diff --git a/src/clientiface.cpp b/src/clientiface.cpp
index d4efe60ef50451bbbf4438bc1564bea47d783893..a6ab1ef41ceb4487d86277bfbbc8b6b89fb22dca 100644 (file)
--- a/src/clientiface.cpp
+++ b/src/clientiface.cpp
@@ -38,10 +38,12 @@ const char *ClientInterface::statenames[] = {
        "Disconnecting",
        "Denied",
        "Created",
-       "InitSent",
+       "AwaitingInit2",
+       "HelloSent",
        "InitDone",
        "DefinitionsSent",
-       "Active"
+       "Active",
+       "SudoMode",
 };
 
 

diff --git a/src/clientiface.h b/src/clientiface.h
index 9be8bda617a652e2543440782af9fbe47023b10b..0d2bca1965ce535ce3bed7fb63fe2c2c51e7dbdc 100644 (file)
--- a/src/clientiface.h
+++ b/src/clientiface.h
@@ -167,6 +167,9 @@ namespace con {
 
 #define CI_ARRAYSIZE(a) (sizeof(a) / sizeof((a)[0]))
 
+// Also make sure to update the ClientInterface::statenames
+// array when modifying these enums
+
 enum ClientState
 {
        CS_Invalid,