diff options
author | Haru <haru@dotalux.com> | 2013-09-21 03:47:19 +0200 |
---|---|---|
committer | Haru <haru@dotalux.com> | 2013-09-21 03:50:26 +0200 |
commit | 51241f66ad6e78cf56b91c41e524c00a1d5abeb2 (patch) | |
tree | 0b03082e62425d126bbff387f09cad4ab0f2bd24 /src/map/chrif.c | |
parent | 556834d0508101b70660c5904b24adacacced3e0 (diff) | |
download | hercules-51241f66ad6e78cf56b91c41e524c00a1d5abeb2.tar.gz hercules-51241f66ad6e78cf56b91c41e524c00a1d5abeb2.tar.bz2 hercules-51241f66ad6e78cf56b91c41e524c00a1d5abeb2.tar.xz hercules-51241f66ad6e78cf56b91c41e524c00a1d5abeb2.zip |
Fixed a potentially negative array subscript causing a mapserver crash
- Fixes bugreport:7717, special thanks to quesoph
http://hercules.ws/board/tracker/issue-7717-map-crash-when-warping-on-a-floating-npc/
- Also changed various map[foo].index to the map_id2index(foo) macro.
Signed-off-by: Haru <haru@dotalux.com>
Diffstat (limited to 'src/map/chrif.c')
-rw-r--r-- | src/map/chrif.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/map/chrif.c b/src/map/chrif.c index c79cf9e4a..53ef808a0 100644 --- a/src/map/chrif.c +++ b/src/map/chrif.c @@ -348,7 +348,7 @@ int chrif_sendmap(int fd) { WFIFOHEAD(fd, 4 + instance->start_id * 4); WFIFOW(fd,0) = 0x2afa; for(i = 0; i < instance->start_id; i++) - WFIFOW(fd,4+i*4) = map[i].index; + WFIFOW(fd,4+i*4) = map_id2index(i); WFIFOW(fd,2) = 4 + i * 4; WFIFOSET(fd,WFIFOW(fd,2)); |