diff options
author | Strangerke | 2014-05-31 17:30:15 +0200 |
---|---|---|
committer | Strangerke | 2014-05-31 17:30:53 +0200 |
commit | 659b7bfeed3b7cc0ccb70fcb41d638a3e8ba3d45 (patch) | |
tree | dbc7b5441b8a53694c26a82f4fc7f1596b2d7d47 /engines/cruise/function.cpp | |
parent | 283db7d769eb4cdd2014bd09132e7b06d17e2f4c (diff) | |
download | scummvm-rg350-659b7bfeed3b7cc0ccb70fcb41d638a3e8ba3d45.tar.gz scummvm-rg350-659b7bfeed3b7cc0ccb70fcb41d638a3e8ba3d45.tar.bz2 scummvm-rg350-659b7bfeed3b7cc0ccb70fcb41d638a3e8ba3d45.zip |
CRUISE: Replace some more strcpy
Diffstat (limited to 'engines/cruise/function.cpp')
-rw-r--r-- | engines/cruise/function.cpp | 27 |
1 files changed, 9 insertions, 18 deletions
diff --git a/engines/cruise/function.cpp b/engines/cruise/function.cpp index 243a9f0a61..327dec4d6c 100644 --- a/engines/cruise/function.cpp +++ b/engines/cruise/function.cpp @@ -335,20 +335,15 @@ int16 Op_RemoveMessage() { } int16 Op_FindSet() { - int16 i; - char name[36] = ""; - char *ptr; - - ptr = (char *) popPtr(); - - if (!ptr) { + char *ptr = (char *) popPtr(); + if (!ptr) return -1; - } - strcpy(name, ptr); + char name[36] = ""; + Common::strlcpy(name, ptr, sizeof(name)); strToUpper(name); - for (i = 0; i < NUM_FILE_ENTRIES; i++) { + for (int i = 0; i < NUM_FILE_ENTRIES; i++) { if (!strcmp(name, filesDatabase[i].subData.name)) { return (i); } @@ -409,14 +404,10 @@ int16 Op_FreeOverlay() { int16 Op_FindProc() { char name[36] = ""; - char *ptr; - int param; - ptr = (char *)popPtr(); - - strcpy(name, ptr); - - param = getProcParam(popVar(), 20, name); + char *ptr = (char *)popPtr(); + Common::strlcpy(name, ptr, sizeof(name)); + int param = getProcParam(popVar(), 20, name); return param; } @@ -572,7 +563,7 @@ int16 Op_LoadAbs() { slot = popVar(); if ((slot >= 0) && (slot < NUM_FILE_ENTRIES)) { - strcpy(name, ptr); + Common::strlcpy(name, ptr, sizeof(name)); strToUpper(name); gfxModuleData_gfxWaitVSync(); |