Fix potential gamepad name string copy overflow

This commit is contained in:
asdqwe 2025-01-17 08:32:54 -03:00
parent 0275d13aa8
commit 72ad7edcae

View File

@ -1652,7 +1652,7 @@ int InitPlatform(void)
// Retrieve gamepad names // Retrieve gamepad names
for (int i = 0; i < MAX_GAMEPADS; i++) for (int i = 0; i < MAX_GAMEPADS; i++)
{ {
if (glfwJoystickPresent(i)) strcpy(CORE.Input.Gamepad.name[i], glfwGetJoystickName(i)); if (glfwJoystickPresent(i)) snprintf(CORE.Input.Gamepad.name[i], sizeof(CORE.Input.Gamepad.name[i]), "%s", glfwGetJoystickName(i));
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
@ -1915,7 +1915,7 @@ static void JoystickCallback(int jid, int event)
{ {
if (event == GLFW_CONNECTED) if (event == GLFW_CONNECTED)
{ {
strcpy(CORE.Input.Gamepad.name[jid], glfwGetJoystickName(jid)); snprintf(CORE.Input.Gamepad.name[jid], sizeof(CORE.Input.Gamepad.name[jid]), "%s", glfwGetJoystickName(jid));
} }
else if (event == GLFW_DISCONNECTED) else if (event == GLFW_DISCONNECTED)
{ {