aboutsummaryrefslogtreecommitdiff
path: root/engines/cruise/function.cpp
diff options
context:
space:
mode:
authorStrangerke2014-05-31 17:30:15 +0200
committerStrangerke2014-05-31 17:30:53 +0200
commit659b7bfeed3b7cc0ccb70fcb41d638a3e8ba3d45 (patch)
treedbc7b5441b8a53694c26a82f4fc7f1596b2d7d47 /engines/cruise/function.cpp
parent283db7d769eb4cdd2014bd09132e7b06d17e2f4c (diff)
downloadscummvm-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.cpp27
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();