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.
minetest · Aug 31, 2015 · 2c637ce · 2c637ce
1 parent 9c44aac
commit 2c637ce
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git 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
@@ -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,