pa/eqemu-server
2
0
Fork 0
mirror of https://github.com/EQEmu/Server.git synced 2025-12-12 01:11:29 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

* Added test for null output of GetItem and log & return if the item cannot be found.

Browse Source
This commit is contained in:
Daerath 2018-01-21 15:49:15 -05:00
parent 008ac97a64
commit 518a5e8ab1
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
zone/client_packet.cpp
View File

@ -11697,7 +11697,14 @@ void Client::Handle_OP_RecipesFavorite(const EQApplicationPacket *app)
}
else {
containers += StringFormat(" in (%u, %u) ", tsf->object_type, tsf->some_id); // container in inventory
combineObjectSlots = database.GetItem(tsf->some_id)->BagSlots;
auto item = database.GetItem(tsf->some_id);
if (!item)
{
Log(Logs::General, Logs::Error, "Invalid container ID: %d. GetItem returned null.\n", tsf->some_id);
return;
}
combineObjectSlots = item->BagSlots;
}
std::string favoriteIDs; //gotta be big enough for 500 IDs
@ -11762,7 +11769,14 @@ void Client::Handle_OP_RecipesSearch(const EQApplicationPacket *app)
else {
// container in inventory
snprintf(containers, 29, "in (%u,%u)", rss->object_type, rss->some_id);
combineObjectSlots = database.GetItem(rss->some_id)->BagSlots;
auto item = database.GetItem(rss->some_id);
if (!item)
{
Log(Logs::General, Logs::Error, "Invalid container ID: %d. GetItem returned null.\n", rss->some_id);
return;
}
combineObjectSlots = item->BagSlots;
}
std::string searchClause;