Ginger88895
/
warzone2100
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add several sanity checks for network packets received. 

git-svn-id: svn+ssh://svn.gna.org/svn/warzone/trunk@3790 4a71c877-e1ca-e34f-864e-861f7616d084
master
Per Inge Mathisen 2008-02-15 21:31:23 +00:00
parent 4c64fcfe8e
commit aa12b3b9e8
2 changed files with 36 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/multiplay.c
View File

@ -723,6 +723,11 @@ BOOL recvMessage(void)
NETuint32_t(&player_id);
NETbool(&host); // Added to check for host quit here -- Buggy
NETend();
if (player_id >= MAX_PLAYERS)
{
debug(LOG_ERROR, "Bad NET_LEAVING received, player ID is %d", (int)player_id);
break;
}
MultiPlayerLeave(player_id);
if (host) // host has quit, need to quit too.
{
@ -743,7 +748,11 @@ BOOL recvMessage(void)
// the player that has just responded
NETuint32_t(&player_id);
NETend();
if (player_id >= MAX_PLAYERS)
{
debug(LOG_ERROR, "Bad NET_PLAYERRESPONDING received, ID is %d", (int)player_id);
break;
}
// This player is now with us!
ingame.JoiningInProgress[player_id] = FALSE;
break;
@ -848,6 +857,12 @@ static BOOL recvResearch()
NETuint32_t(&index);
NETend();
if (player >= MAX_PLAYERS || index >= numResearch)
{
debug(LOG_ERROR, "Bad NET_RESEARCH received, player is %d, index is %u", (int)player, index);
return FALSE;
}
pPlayerRes = asPlayerResList[player] + index;
// If they have completed the research
@ -932,6 +947,12 @@ BOOL recvResearchStatus()
NETuint32_t(&index);
NETend();
if (player >= MAX_PLAYERS || index >= numResearch)
{
debug(LOG_ERROR, "Bad NET_RESEARCHSTATUS received, player is %d, index is %u", (int)player, index);
return FALSE;
}
pPlayerRes = asPlayerResList[player] + index;
// psBuilding may be null if finishing

14
src/multisync.c
View File

@ -764,6 +764,13 @@ BOOL recvStructureCheck()
NETuint16_t(&z);
NETfloat(&direction);
if (player >= MAX_PLAYERS)
{
debug(LOG_ERROR, "Bad NET_CHECK_STRUCT received!");
NETend();
return FALSE;
}
// If the structure exists our job is easy
pS = IdToStruct(ref, player);
if (pS)
@ -1042,7 +1049,6 @@ BOOL sendScoreCheck(void)
}
BOOL recvScoreSubmission()
{
uint8_t player;
@ -1175,6 +1181,12 @@ BOOL recvPing()
NETbool(&isNew);
NETend();
if (sender >= MAX_PLAYERS)
{
debug(LOG_ERROR, "Bad NET_PING packet, sender is %d", (int)sender);
return FALSE;
}
// If this is a new ping, respond to it
if (isNew)
{