summaryrefslogtreecommitdiff
path: root/src/map/mail.c
diff options
context:
space:
mode:
authorHaruna <haru@dotalux.com>2015-09-12 18:36:36 +0200
committerHaruna <haru@dotalux.com>2015-09-12 18:36:36 +0200
commitd8caa7c8e7296a87072b6775902b985a1faebadf (patch)
treeddc6cdaf9ff81c01033e3d390c40ce0432edafaf /src/map/mail.c
parentc76aa5f4843a521b1839875bf6c139f4658bd6d1 (diff)
parent8fc98e3906f38b25c71b60b26bb31d0027ee02a0 (diff)
downloadhercules-d8caa7c8e7296a87072b6775902b985a1faebadf.tar.gz
hercules-d8caa7c8e7296a87072b6775902b985a1faebadf.tar.bz2
hercules-d8caa7c8e7296a87072b6775902b985a1faebadf.tar.xz
hercules-d8caa7c8e7296a87072b6775902b985a1faebadf.zip
Merge pull request #712 from 4144/mailchecks
Add extra checks into mail system.
Diffstat (limited to 'src/map/mail.c')
-rw-r--r--src/map/mail.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/map/mail.c b/src/map/mail.c
index 0bdad2877..8acfbdcb9 100644
--- a/src/map/mail.c
+++ b/src/map/mail.c
@@ -83,7 +83,7 @@ unsigned char mail_setitem(struct map_session_data *sd, int idx, int amount) {
if( idx < 0 || idx >= MAX_INVENTORY )
return 1;
- if( amount < 0 || amount > sd->status.inventory[idx].amount )
+ if( amount <= 0 || amount > sd->status.inventory[idx].amount )
return 1;
if( !pc_can_give_items(sd) || sd->status.inventory[idx].expire_time ||
!itemdb_canmail(&sd->status.inventory[idx],pc_get_group_level(sd)) ||
@@ -122,6 +122,8 @@ bool mail_setattachment(struct map_session_data *sd, struct mail_message *msg)
memcpy(&msg->item, &sd->status.inventory[n], sizeof(struct item));
msg->item.amount = sd->mail.amount;
+ if (msg->item.amount != sd->mail.amount) // check for amount overflow
+ return false;
}
else
memset(&msg->item, 0x00, sizeof(struct item));