Commit 3984c7d8e377cbd000fe1fdb166ed043ba1cda62

Ryan C. Gordon 2013-08-20T23:20:32

Actually, this should be a memcpy(). We already know the exact length we just allocated, and we plan to append our own null terminator to the end of the copy, so this makes more sense.

1
2
3
4
5
6
7
8
9
10
11
12
13
diff --git a/src/filesystem/beos/SDL_sysfilesystem.cc b/src/filesystem/beos/SDL_sysfilesystem.cc
index 06b1880..decf84c 100644
--- a/src/filesystem/beos/SDL_sysfilesystem.cc
+++ b/src/filesystem/beos/SDL_sysfilesystem.cc
@@ -63,7 +63,7 @@ SDL_GetBasePath(void)
         return NULL;
     }
 
-    strcpy(retval, str);
+    SDL_memcpy(retval, str, len);
     retval[len] = '/';
     retval[len+1] = '\0';
     return retval;