rhotator crash #6502
Labels
No Label
1. kind/balancing
1. kind/breaking
1. kind/bug
1. kind/construction
1. kind/documentation
1. kind/enhancement
1. kind/griefing
1. kind/invalid
1. kind/meme
1. kind/node limit
1. kind/other
1. kind/protocol
2. prio/controversial
2. prio/critical
2. prio/elevated
2. prio/good first issue
2. prio/interesting
2. prio/low
3. source/art
3. source/client
3. source/engine
3. source/ingame
3. source/integration
3. source/lag
3. source/license
3. source/mod upstream
3. source/unknown
3. source/website
4. step/approved
4. step/at work
4. step/blocked
4. step/discussion
4. step/help wanted
4. step/needs confirmation
4. step/partially fixed
4. step/question
4. step/ready to deploy
4. step/ready to QA test
4. step/want approval
5. result/cannot reproduce
5. result/duplicate
5. result/fixed
5. result/maybe
5. result/wontfix
ugh/petz
ugh/QA main
ugh/QA NOK
ugh/QA OK
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: your-land/bugtracker#6502
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
meaning, it can fail and returned ID needs to be checked
opened upstream issue:
https://github.com/SwissalpS/replacer/issues/52wrong repo, it's not replacer, it's rhotator!Did we run out of HUD ids?!They are stored in
std::vector<HudElement *> hud;
per-player.and
hud_add()
checks if ID of the new element does not fit intou32
(addHud
also seems to re-use freed ids):I guess we can't exclude possibility of some mod leaking HUDs...It could be any other mod besides replacer, so we need to find the player who used the replacer when server has crashed and then try to figure out what they were doing that could've created HUDs in excess.And in the log is no coord where?
replacer crashto rhotator crashrhotator
has a "memory" function that tries to rotate blocks after placement even if you place them by hand or with a replacer...This crash seems to be fixed upstream:3b29e5f438
And we have merged it... does it not work?UPD:
seems like needs to guard other place too..
Something weird is going on there.
i can't remember if used to know this and just forgot, but all my hud management stuff seems to already makes sure the id is defined before trying to get the hud definition (and then validates it further before making any changes). sometimes minetest's API makes it really easy for things to fail and mess up other code's logic - HUD management is one of the worst examples.