-
-
Notifications
You must be signed in to change notification settings - Fork 638
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
Fix remove imbuing from item #139
Fix remove imbuing from item #139
Conversation
2021-10-15 14:52:56 - free(): invalid pointer |
I can confirm the crash, sometimes it occurs some time after the imbuing ends. |
before this commit didn't crash? |
i confirm to, i try port this to canary but crashes stop me from fixing it, i have almost done port but with this crash |
This is very strange, in otbr it works and in Canary it doesn't. Since it doesn't change the code |
I think that it may crash in otbr too, but noone actually tested that, I may be able to test in couple hours. |
on canary cyclopedia can stop serwer because we have only 2 monsters in data... maybe this bug can happening becouse of lack of other things |
this problem also happens in the otbr, there is a problem with the fix that causes the server to crash |
in case I posted about it later in the commit. |
I dont know if its known thing here, but initially it (removing both imbues) got broken after this commit: |
Thanks, you solve the problem. for (uint8_t slot = 0; slot < slots; slot++) { and move item->setImbuement(slot, 0); inside: if (duration > 0 && newDuration == 0) { |
@FakeShinoda Yeah it seems fine after quick test of both issues. |
Co-Authored-By: Rick <23365432+FakeShinoda@users.noreply.github.com>
bae2066
to
23d1bdc
Compare
Kudos, SonarCloud Quality Gate passed! |
Fixed on: #150 |
Description
Fix remove imbuing from item
Behaviour
Actual
When one imbuing end, all imbuing from item will be removed
Expected
Only one imbuing removed
Fixes
opentibiabr/otservbr-global-archived#2649
Type of change
Please delete options that are not relevant.