thirdpartymirrors/RingRacers
1
0
Fork 0
mirror of https://github.com/KartKrewDev/RingRacers.git synced 2026-04-26 12:01:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'fix-admin-commands' into 'master'

Browse source 
Put bounds checking on promote and demote commands

See merge request KartKrew/Kart!1916
This commit is contained in:
James R. 2024-02-20 04:20:44 +00:00
commit 36b4e88e11
1 changed files with 10 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
src/d_netcmd.c
View file

@ -3864,10 +3864,6 @@ void RemoveAdminPlayer(INT32 playernum)
static void Command_Verify_f(void) static void Command_Verify_f(void)
{ {
char buf[8]; // Should be plenty
char *temp;
INT32 playernum;
if (client) if (client)
{ {
CONS_Printf(M_GetText("Only the server can use this.\n")); CONS_Printf(M_GetText("Only the server can use this.\n"));
@ -3886,16 +3882,13 @@ static void Command_Verify_f(void)
return; return;
} }
strlcpy(buf, COM_Argv(1), sizeof (buf)); INT32 playernum = atoi(COM_Argv(1));
playernum = atoi(buf); if (playernum >= 0 && playernum < MAXPLAYERS && playeringame[playernum])
{
temp = buf; UINT8 buf[1] = {playernum};
WRITEUINT8(temp, playernum);
if (playeringame[playernum])
SendNetXCmd(XD_VERIFIED, buf, 1); SendNetXCmd(XD_VERIFIED, buf, 1);
}
} }
static void Got_Verification(const UINT8 **cp, INT32 playernum) static void Got_Verification(const UINT8 **cp, INT32 playernum)
@ -3920,10 +3913,6 @@ static void Got_Verification(const UINT8 **cp, INT32 playernum)
static void Command_RemoveAdmin_f(void) static void Command_RemoveAdmin_f(void)
{ {
char buf[8]; // Should be plenty
char *temp;
INT32 playernum;
if (client) if (client)
{ {
CONS_Printf(M_GetText("Only the server can use this.\n")); CONS_Printf(M_GetText("Only the server can use this.\n"));
@ -3936,16 +3925,13 @@ static void Command_RemoveAdmin_f(void)
return; return;
} }
strlcpy(buf, COM_Argv(1), sizeof(buf)); INT32 playernum = atoi(COM_Argv(1));
playernum = atoi(buf); if (playernum >= 0 && playernum < MAXPLAYERS && playeringame[playernum])
{
temp = buf; UINT8 buf[1] = {playernum};
WRITEUINT8(temp, playernum);
if (playeringame[playernum])
SendNetXCmd(XD_DEMOTED, buf, 1); SendNetXCmd(XD_DEMOTED, buf, 1);
}
} }
static void Got_Removal(const UINT8 **cp, INT32 playernum) static void Got_Removal(const UINT8 **cp, INT32 playernum)