diff options
| author | Torbjörn Andersson | 2018-07-21 09:20:25 +0200 |
|---|---|---|
| committer | Torbjörn Andersson | 2018-07-21 09:23:06 +0200 |
| commit | 04585b60ec8ffc406febd09a45c7512057be3fa8 (patch) | |
| tree | bf81d6cc8c4826ed8cc92cea3fed3ae593bfb47a /engines/agos | |
| parent | b8abe400850a23d12fe5cdc24d7106820d0f13fd (diff) | |
| download | scummvm-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.
Diffstat (limited to 'engines/agos')
| -rw-r--r-- | engines/agos/animation.cpp | 4 |
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); |