nethack-gwaa-kiwi/nethack
1
0
Fork 0
Code Pull Requests Activity

Fix: impossible from splitting named stack on bill

Browse Source 
Trying to split an unpaid stack of named items in a shop, with
perm_invent enabled, would cause an impossible 'unpaid_cost: object
wasn't on any bill' because copy_oextra -> oname triggered an inventory
update while the newly created split stack was marked unpaid but before
the billing information had been split to match.  Defer the copy_oextra
call until the billing info has already been split.
This commit is contained in:
Michael Meyer
2022-09-16 22:50:39 -04:00
committed by PatR
parent 8ef9cf85ab
commit 7e96026b44
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/mkobj.c
View File

@@ -423,11 +423,11 @@ splitobj(struct obj *obj, long num)
be tracking the original */ be tracking the original */
if (otmp->where == OBJ_LUAFREE) if (otmp->where == OBJ_LUAFREE)
otmp->where = OBJ_FREE; otmp->where = OBJ_FREE;
if (obj->unpaid)
splitbill(obj, otmp);
copy_oextra(otmp, obj); copy_oextra(otmp, obj);
if (has_omid(otmp)) if (has_omid(otmp))
free_omid(otmp); /* only one association with m_id*/ free_omid(otmp); /* only one association with m_id*/
if (obj->unpaid)
splitbill(obj, otmp);
if (obj->timed) if (obj->timed)
obj_split_timers(obj, otmp); obj_split_timers(obj, otmp);
if (obj_sheds_light(obj)) if (obj_sheds_light(obj))