Rev 127 Zone crash after looting an item

[Derision]

char *link;
sprintf(link, "%c%06X%s%s%c", ...

You are declaring link as a pointer to an array of characters, but not initialising it to a valid value, so it will point to a random position in memory.

You should declare it like char link[100]; (replacing 100 with whatever the maximum size of a link is).

[paaco]

I'm no coding genius, so this may not be helpful at all. But it appears to be the part that Derision just mentioned. This is what pops up while zone is compiling and it gets to that part.

1>c:\eq\eqemuserver\zone\playercorpse.cpp(1005) : warning C4700: uninitialized local variable 'link' used

[AndMetal]

Originally Posted by Derision

You should declare it like char link[100]; (replacing 100 with whatever the maximum size of a link is).

My concern is that the link's length depends on the name of the item, since it is appended on the end, just before the terminating character 0x12. However, an item name should be a max of 64, so we could just set the length to 1+6+39+64+1 = 111.

I'll go ahead and compile the changes & see what happens.

Edit: we could use a string, like was done before, but it would be so much more of a pita to get it back to a predictable char.

[Derision]

I'm not sure if your calculation include the `\0` terminator that sprintf will automatically put at the end of the string. If not, you should add 1 byte for that.

[AndMetal]

I started with 111 (didn't realize the last byte added by sprintf at the time) and it works fine, primarily because the longest item name in the DB is 50 characters long:

mysql> SELECT id, Name, CHAR_LENGTH(Name) as Name_length FROM items ORDER BY Name_length DESC, id ASC LIMIT 5;
+-------+----------------------------------------------------+-------------+
| id    | Name                                               | Name_length |
+-------+----------------------------------------------------+-------------+
| 18557 | Prophecy of Vah: A History of the Vah Shir Vol.III |          50 |
| 20429 | Banner Material Kit: Focus of Benefit Conservation |          50 |
| 18556 | Prophecy of Vah: A History of the Vah Shir Vol.II  |          49 |
| 46814 | Gnomework Model XVII's Experimental Plate Backing  |          49 |
| 57937 | First Sisters of the Blackfeather Harpies Vol. II  |          49 |
+-------+----------------------------------------------------+-------------+
5 rows in set (0.98 sec)

I'll get this updated into SVN once I have a chance to finish some other stuff (unless someone wants to take care of it first).

[paaco]

Thanks for the fix Andmetal, took care of the problem I was having. I can now loot with Rev 129