aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTorbjörn Andersson2018-07-21 09:20:25 +0200
committerTorbjörn Andersson2018-07-21 09:23:06 +0200
commit04585b60ec8ffc406febd09a45c7512057be3fa8 (patch)
treebf81d6cc8c4826ed8cc92cea3fed3ae593bfb47a
parentb8abe400850a23d12fe5cdc24d7106820d0f13fd (diff)
downloadscummvm-rg350-04585b60ec8ffc406febd09a45c7512057be3fa8.tar.gz
scummvm-rg350-04585b60ec8ffc406febd09a45c7512057be3fa8.tar.bz2
scummvm-rg350-04585b60ec8ffc406febd09a45c7512057be3fa8.zip
AGOS: Silence GCC 8 warning
Since 'filename' is 'baseName', plus a file extension, make the 'filename' buffer larger than the 'baseName' buffer. Even though this almost certainly means that the 'filename' buffer is now quite a bit larger than it needs to be, I don't think there's any reason to quibble about it.
-rw-r--r--engines/agos/animation.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/engines/agos/animation.cpp b/engines/agos/animation.cpp
index 123e5805dd..e374c14701 100644
--- a/engines/agos/animation.cpp
+++ b/engines/agos/animation.cpp
@@ -519,7 +519,7 @@ bool MoviePlayerSMK::processFrame() {
MoviePlayer *makeMoviePlayer(AGOSEngine_Feeble *vm, const char *name) {
char baseName[40];
- char filename[20];
+ char filename[45];
int baseLen = strlen(name) - 4;
memset(baseName, 0, sizeof(baseName));
@@ -528,7 +528,7 @@ MoviePlayer *makeMoviePlayer(AGOSEngine_Feeble *vm, const char *name) {
if (vm->getLanguage() == Common::DE_DEU && baseLen >= 8) {
// Check short filename to work around
// bug in a German Windows 2CD version.
- char shortName[20];
+ char shortName[10];
memset(shortName, 0, sizeof(shortName));
memcpy(shortName, baseName, 6);