teamnwah/openmw-tes3coop
2
0
Fork 1
forked from mirror/openmw-tes3mp
Code Issues Pull requests 1 Releases Wiki Activity

Fix a potential exploit that made it possible to reset the owner of any item

Browse source
This commit is contained in:
scrawl 2014-05-15 04:20:40 +02:00
parent a6e1d7ffd6
commit bac4d875df
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
apps/openmw/mwgui/inventoryitemmodel.cpp
View file

@ -64,7 +64,10 @@ void InventoryItemModel::moveItem(const ItemStack &item, size_t count, ItemModel
bool setNewOwner = false; bool setNewOwner = false;
// Are you dead? Then you wont need that anymore // Are you dead? Then you wont need that anymore
if (mActor.getClass().isActor() && mActor.getClass().getCreatureStats(mActor).isDead()) if (mActor.getClass().isActor() && mActor.getClass().getCreatureStats(mActor).isDead()
// Make sure that the item is actually owned by the dead actor
// Prevents a potential exploit for resetting the owner of any item, by placing the item in a corpse
&& Misc::StringUtils::ciEqual(item.mBase.getCellRef().mOwner, mActor.getCellRef().mRefID))
setNewOwner = true; setNewOwner = true;
otherModel->copyItem(item, count, setNewOwner); otherModel->copyItem(item, count, setNewOwner);