diff options
author | Max Horn | 2009-01-16 01:48:55 +0000 |
---|---|---|
committer | Max Horn | 2009-01-16 01:48:55 +0000 |
commit | 42c46aa5a1e570b3e8d0d857e6ef9291d8f20a4f (patch) | |
tree | 30c8526f2019d3046e745d1b2d4a8539bd7b944c /engines | |
parent | 7d3c94a9caa0da1fd9fa5e182024a3c2b43b3ccb (diff) | |
download | scummvm-rg350-42c46aa5a1e570b3e8d0d857e6ef9291d8f20a4f.tar.gz scummvm-rg350-42c46aa5a1e570b3e8d0d857e6ef9291d8f20a4f.tar.bz2 scummvm-rg350-42c46aa5a1e570b3e8d0d857e6ef9291d8f20a4f.zip |
Mark cruise save/load code as not endian safe
svn-id: r35869
Diffstat (limited to 'engines')
-rw-r--r-- | engines/cruise/saveload.cpp | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/engines/cruise/saveload.cpp b/engines/cruise/saveload.cpp index 3d8321e420..e8467c327b 100644 --- a/engines/cruise/saveload.cpp +++ b/engines/cruise/saveload.cpp @@ -318,6 +318,8 @@ void loadScriptsFromSave(Common::InSaveFile& currentSaveFile, scriptInstanceStru if (ptr->varA) { ptr->var6 = (uint8 *) mallocAndZero(ptr->varA); + // FIXME: This code is not endian safe, and breaks if struct + // packing changes. Read/write the members one by one instead. currentSaveFile.read(ptr->var6, ptr->varA); } @@ -441,6 +443,8 @@ void saveCT(Common::OutSaveFile& currentSaveFile) { if(numberOfWalkboxes) { + // FIXME: This code is not endian safe, and breaks if struct + // packing changes. Read/write the members one by one instead. currentSaveFile.write(walkboxColor, numberOfWalkboxes * 2); currentSaveFile.write(walkboxState, numberOfWalkboxes * 2); } @@ -473,6 +477,8 @@ void loadSavegameDataSub6(Common::InSaveFile& currentSaveFile) { numberOfWalkboxes = currentSaveFile.readUint16LE(); if (numberOfWalkboxes) { + // FIXME: This code is not endian safe, and breaks if struct + // packing changes. Read/write the members one by one instead. currentSaveFile.read(walkboxColor, numberOfWalkboxes * 2); currentSaveFile.read(walkboxState, numberOfWalkboxes * 2); } @@ -481,6 +487,8 @@ void loadSavegameDataSub6(Common::InSaveFile& currentSaveFile) { persoTable[i] = (persoStruct*)currentSaveFile.readSint32LE(); if (persoTable[i]) { + // FIXME: This code is not endian safe, and breaks if struct + // packing changes. Read/write the members one by one instead. assert(sizeof(persoStruct) == 0x6AA); persoTable[i] = (persoStruct *)mallocAndZero(sizeof(persoStruct)); currentSaveFile.read(persoTable[i], 0x6AA); @@ -736,8 +744,10 @@ int loadSavegameData(int saveGameIdx) { var41 = currentSaveFile->readSint16LE(); entrerMenuJoueur = currentSaveFile->readSint16LE(); + // FIXME: This code is not endian safe, and breaks if struct + // packing changes. Read/write the members one by one instead. currentSaveFile->read(newPal, sizeof(int16) * NBCOLORS); - currentSaveFile->read(newPal, sizeof(int16) * NBCOLORS); + currentSaveFile->read(newPal, sizeof(int16) * NBCOLORS); // FIXME: Should this read into workpal ? // here code seems bogus... this should read music name and it may be a buffer overrun currentSaveFile->skip(21); @@ -751,6 +761,8 @@ int loadSavegameData(int saveGameIdx) { currentSaveFile->read(backgroundTable[i].extention, 6); } + // FIXME: This code is not endian safe, and breaks if struct + // packing changes. Read/write the members one by one instead. currentSaveFile->read(palScreen, sizeof(int16) * NBCOLORS * NBSCREENS); currentSaveFile->read(initVar5, 24); currentSaveFile->read(globalVars, stateID * 2); // ok |