Issue information

Issue ID
#1573
Status
Working as Intended
Severity
None
Started
Hercules Elf Bot
May 26, 2008 21:36
Last Post
Hercules Elf Bot
May 26, 2008 21:36
Confirmation
N/A

Hercules Elf Bot - May 26, 2008 21:36

Originally posted by [b]Everade[/b]
http://www.eathena.ws/board/index.php?autocom=bugtracker&showbug=1573

I haven't made the fixes, but they worked fine for me.

status.c
crash on line 4191 [fix see diff]
crash on line 4224
CODE
Index: status.c
===================================================================
--- status.c    (revision 12718)
+++ status.c    (working copy)
@@ -4188,6 +4188,8 @@
        return ((TBL_PC*)bl)->status.guild_id;
    case BL_PET:
        if (((TBL_PET*)bl)->msd)
+        if( !bl || !((TBL_PET*)bl)->msd)
+        return 0;
            return ((TBL_PET*)bl)->msd->status.guild_id;
        break;
    case BL_MOB:



pc.c
crash on line 6494 [fix see diff]
Just saw now that RedXIII reported it allready here:
http://www.eathena.ws/board/index.php?auto...mp;showbug=1568
CODE
Index: pc.c
===================================================================
--- pc.c    (revision 12718)
+++ pc.c    (working copy)
@@ -6481,6 +6481,11 @@
    int i;
    nullpo_retr(0, sd);

+    if(n < 0 || n >= MAX_INVENTORY) {
+        clif_unequipitemack(sd,0,0,0);
+        return 0;
+    }
+
// -- moonsoul    (if player is berserk then cannot unequip)
//
    if(!(flag&2) && sd->sc.count && (sd->sc.data[SC_BLADESTOP] || sd->sc.data[SC_BERSERK])){
@@ -6488,6 +6493,7 @@
        return 0;
    }

+
    if(battle_config.battle_log)
        ShowInfo("unequip %d %x:%x\n",n,pc_equippoint(sd,n),sd->status.inventory[n].equip);


This post has been edited by Everade: May 26 2008, 06:10 PM