aboutsummaryrefslogtreecommitdiff
path: root/src/server/collision/Management/MMapManager.cpp
diff options
context:
space:
mode:
authorSpp <spp@jorge.gr>2013-01-09 13:13:39 +0100
committerSpp <spp@jorge.gr>2013-01-09 13:13:39 +0100
commit5f15600e7a26170f0aacc77e9417df61db45a992 (patch)
tree0bfbe55a73dfb7bf5ab66202ab42079166c98dc8 /src/server/collision/Management/MMapManager.cpp
parentd36d2eac768ff3a7761d55455edd85656bc6662b (diff)
Core/Build: Fix linking and sooo many compile warnings
Diffstat (limited to 'src/server/collision/Management/MMapManager.cpp')
-rw-r--r--src/server/collision/Management/MMapManager.cpp14
1 files changed, 9 insertions, 5 deletions
diff --git a/src/server/collision/Management/MMapManager.cpp b/src/server/collision/Management/MMapManager.cpp
index c7dea358d92..1b6cf5c0260 100644
--- a/src/server/collision/Management/MMapManager.cpp
+++ b/src/server/collision/Management/MMapManager.cpp
@@ -52,8 +52,14 @@ namespace MMAP
}
dtNavMeshParams params;
- fread(&params, sizeof(dtNavMeshParams), 1, file);
+ int count = fread(&params, sizeof(dtNavMeshParams), 1, file);
fclose(file);
+ if (count != 1)
+ {
+ sLog->outDebug(LOG_FILTER_MAPS, "MMAP:loadMapData: Error: Could not read params from file '%s'", fileName);
+ delete [] fileName;
+ return false;
+ }
dtNavMesh* mesh = dtAllocNavMesh();
ASSERT(mesh);
@@ -82,7 +88,7 @@ namespace MMAP
return uint32(x << 16 | y);
}
- bool MMapManager::loadMap(const std::string& basePath, uint32 mapId, int32 x, int32 y)
+ bool MMapManager::loadMap(const std::string& /*basePath*/, uint32 mapId, int32 x, int32 y)
{
// make sure the mmap is loaded and ready to load tiles
if (!loadMapData(mapId))
@@ -114,9 +120,7 @@ namespace MMAP
// read header
MmapTileHeader fileHeader;
- fread(&fileHeader, sizeof(MmapTileHeader), 1, file);
-
- if (fileHeader.mmapMagic != MMAP_MAGIC)
+ if (fread(&fileHeader, sizeof(MmapTileHeader), 1, file) != 1 || fileHeader.mmapMagic != MMAP_MAGIC)
{
sLog->outError(LOG_FILTER_MAPS, "MMAP:loadMap: Bad header in mmap %03u%02i%02i.mmtile", mapId, x, y);
return false;