diff options
author | rubidium <rubidium@openttd.org> | 2013-11-23 13:15:07 +0000 |
---|---|---|
committer | rubidium <rubidium@openttd.org> | 2013-11-23 13:15:07 +0000 |
commit | 0e9c9921040a1d0e2aa4b820b20535f40a0d75a3 (patch) | |
tree | 6dd17568acbfb75610e9b4e248436426ead79e89 /src/saveload/oldloader.cpp | |
parent | b3e93d65208f74802595b12e682d98a4d534a328 (diff) | |
download | openttd-0e9c9921040a1d0e2aa4b820b20535f40a0d75a3.tar.xz |
(svn r26058) -Fix: handle the return value of a number of functions better
Diffstat (limited to 'src/saveload/oldloader.cpp')
-rw-r--r-- | src/saveload/oldloader.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/saveload/oldloader.cpp b/src/saveload/oldloader.cpp index f6ebb48bc..ff911a899 100644 --- a/src/saveload/oldloader.cpp +++ b/src/saveload/oldloader.cpp @@ -250,10 +250,9 @@ static SavegameType DetermineOldSavegameType(FILE *f, char *title, const char *l /* Can't fseek to 0 as in tar files that is not correct */ long pos = ftell(f); - if (!CheckOldSavegameType(f, temp, lastof(temp), TTO_HEADER_SIZE)) { + if (pos >= 0 && !CheckOldSavegameType(f, temp, lastof(temp), TTO_HEADER_SIZE)) { type = SGT_TTD; - fseek(f, pos, SEEK_SET); - if (!CheckOldSavegameType(f, temp, lastof(temp), TTD_HEADER_SIZE)) { + if (fseek(f, pos, SEEK_SET) < 0 || !CheckOldSavegameType(f, temp, lastof(temp), TTD_HEADER_SIZE)) { type = SGT_INVALID; } } |