Browse Source

Fixed potentially overlapping memcpy() to use memmove()

(cherry picked from commit 91b76f5c297bcf21463852d02b0a6b2a73e5db59)
Sam Lantinga 3 months ago
parent
commit
5d3a382eb7
2 changed files with 2 additions and 2 deletions
  1. 1 1
      src/joystick/hidapi/SDL_hidapi_ps4.c
  2. 1 1
      src/joystick/hidapi/SDL_hidapi_ps5.c

+ 1 - 1
src/joystick/hidapi/SDL_hidapi_ps4.c

@@ -263,7 +263,7 @@ static SDL_bool HIDAPI_DriverPS4_InitDevice(SDL_HIDAPI_Device *device)
         j = -1;
         for (i = 0; i < 12; i += 2) {
             j += 1;
-            SDL_memcpy(&serial[j], &device->serial[i], 2);
+            SDL_memmove(&serial[j], &device->serial[i], 2);
             j += 2;
             serial[j] = '-';
         }

+ 1 - 1
src/joystick/hidapi/SDL_hidapi_ps5.c

@@ -371,7 +371,7 @@ static SDL_bool HIDAPI_DriverPS5_InitDevice(SDL_HIDAPI_Device *device)
         j = -1;
         for (i = 0; i < 12; i += 2) {
             j += 1;
-            SDL_memcpy(&serial[j], &device->serial[i], 2);
+            SDL_memmove(&serial[j], &device->serial[i], 2);
             j += 2;
             serial[j] = '-';
         }