Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash caused by InvRef:set_list(LISTNAME, nil) in allow* callbacks #13358

Closed
SmallJoker opened this issue Mar 26, 2023 · 0 comments · Fixed by #13360
Closed

Crash caused by InvRef:set_list(LISTNAME, nil) in allow* callbacks #13358

SmallJoker opened this issue Mar 26, 2023 · 0 comments · Fixed by #13360
Labels
Bug Issues that were confirmed to be a bug @ Script API

Comments

@SmallJoker
Copy link
Member

Minetest version

Version 5.7.0-dev baf99f8

Thread 19 "Server" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffb3fff640 (LWP 9688)]
0x0000555555b69a66 in IMoveAction::apply(InventoryManager*, ServerActiveObject*, IGameDef*) ()
(gdb) bt
#0  0x0000555555b69a66 in IMoveAction::apply(InventoryManager*, ServerActiveObject*, IGameDef*) ()
#1  0x000055555593b995 in Server::handleCommand_InventoryAction(NetworkPacket*) ()
#2  0x0000555555c123ac in Server::ProcessData(NetworkPacket*) ()
#3  0x0000555555c125a7 in Server::Receive() ()
#4  0x0000555555c26ef2 in ServerThread::run() ()
OS / Hardware

Linux x86_64

Summary
  • put & take work flawlessly
  • move segfaults (list name detached:PLAYERNAME_workbenchX where X is any of 3,4,5)
Steps to reproduce
  1. https://github.com/cornernote/minetest-workbench/
  2. Place a workbench
  3. Open
  4. Move items
  5. Crash (sometimes it works due to overlapping memory areas)

For reference, #11882 very likely suffers from the same issue.

@SmallJoker SmallJoker added Bug Issues that were confirmed to be a bug @ Script API labels Mar 26, 2023
@SmallJoker SmallJoker changed the title Crash casused by InvRef:set_list(LISTNAME, nil) in allow* callbacks Crash caused by InvRef:set_list(LISTNAME, nil) in allow* callbacks Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues that were confirmed to be a bug @ Script API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant