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

Found a BUG that caused the server to crash.[Bug] #225

Open
ddk2001 opened this issue Mar 30, 2024 · 33 comments
Open

Found a BUG that caused the server to crash.[Bug] #225

ddk2001 opened this issue Mar 30, 2024 · 33 comments
Labels
bug Something isn't working

Comments

@ddk2001
Copy link

ddk2001 commented Mar 30, 2024

🐛 Bug report

Found a BUG, in the copy of Blackstone Abyss, boss's creature_template entry: 9018. When the blood hit about 30%, it appeared: unable to attack in escape state. Then the server crashed. At the same time, there are still many tasks that cannot be completed in dialogue with NPC.
TEK`(6F849G_~_T 8RR6K4O

  • No information

Expected behavior

  • No information

Steps to reproduce

  1. ...
  2. Profit.

Version

Client Version:

  • No information
@ddk2001 ddk2001 added the bug Something isn't working label Mar 30, 2024
@ddk2001 ddk2001 changed the title 🐛 [Bug] Found a BUG that caused the server to crash.[Bug] Mar 30, 2024
@leelf00
Copy link

leelf00 commented Mar 31, 2024

Ok,I will debug it.

@ddk2001
Copy link
Author

ddk2001 commented Mar 31, 2024

Later, I researched it and found out that the problem was that as long as the character was charmed, ran away in fear, transformed into a small animal, etc., the server would go down.

@ddk2001
Copy link
Author

ddk2001 commented Mar 31, 2024

Once this quest has started, horseback riding will cause the server to go down. Several places have been found to cause server crashes. Basically, it is down because the character is not controlled.
MATNJ({KE@6D~Y4$}T0MCK5

@leelf00
Copy link

leelf00 commented Mar 31, 2024

quest id?

@ddk2001
Copy link
Author

ddk2001 commented Mar 31, 2024

14293

@leelf00
Copy link

leelf00 commented Mar 31, 2024

Can you post crash log?

@ddk2001
Copy link
Author

ddk2001 commented Mar 31, 2024

I didn't compile the debug. I'll compile a copy today.

@ddk2001
Copy link
Author

ddk2001 commented Apr 1, 2024

EN_FQ@26WC_5IR(L_M0363T
XQ(GQC@0XB0MZRGK4Z_XYJD
When I log in in DeBug mode, this is prompted.

@leelf00
Copy link

leelf00 commented Apr 1, 2024

Any crash logs from logs folder?

@ddk2001
Copy link
Author

ddk2001 commented Apr 1, 2024

the release is not Any crash logs from logs folder。can you test it?

@ddk2001
Copy link
Author

ddk2001 commented Apr 1, 2024

I'm going to check this debug log on the office computer on Monday.

@ddk2001
Copy link
Author

ddk2001 commented Apr 2, 2024

Application Log
Start: Tue Apr 2 10:05:54 2024


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: (low.x <= high.x) && (low.y <= high.y) && (low.z <= high.z)
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:97


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: ! low.isNaN() && ! high.isNaN()
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:98


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: (low.x <= high.x) && (low.y <= high.y) && (low.z <= high.z)
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:97


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: ! low.isNaN() && ! high.isNaN()
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:98


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: (low.x <= high.x) && (low.y <= high.y) && (low.z <= high.z)
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:97


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: ! low.isNaN() && ! high.isNaN()
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:98


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: (low.x <= high.x) && (low.y <= high.y) && (low.z <= high.z)
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:97


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: ! low.isNaN() && ! high.isNaN()
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:98


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: (low.x <= high.x) && (low.y <= high.y) && (low.z <= high.z)
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:97


Assertion Failure - worldserver.exe
Debug assertion failure

Expression: ! low.isNaN() && ! high.isNaN()
G:\yuanma\4.1ceshi\Legends-of-Azeroth-Pandaria-5.4.8\dep\g3dlite\include\G3D/AABox.h:98

@ddk2001
Copy link
Author

ddk2001 commented Apr 2, 2024

There is only so much for this log file to be recorded. The following error message is not logged.

@leelf00
Copy link

leelf00 commented Apr 2, 2024

Seems gameobject issue

@ddk2001
Copy link
Author

ddk2001 commented Apr 2, 2024

I think it's also a matter of roles. Once the play is controlled, the server goes down.

@ddk2001
Copy link
Author

ddk2001 commented Apr 2, 2024

Did you test it later? Have you found the reason for the downtime? I took a look at gameobject.cpp. I didn't see anything wrong.

@ddk2001
Copy link
Author

ddk2001 commented Apr 3, 2024

Thank you! I'm currently on vacation. I'll try it in about 4 days.

@ddk2001
Copy link
Author

ddk2001 commented May 13, 2024

I found that your source code has the same bug as pandaria, when you buy more than 200,000 gold coins at one time, the gold will not be deducted.

@leelf00
Copy link

leelf00 commented May 13, 2024

Yes.We fork from alexkulya's source and have most same bugs.

@leelf00
Copy link

leelf00 commented May 13, 2024

How to reproduce? use battle coin buy gold coins?

@ddk2001
Copy link
Author

ddk2001 commented May 13, 2024

No, for example, if I buy 2,200 gold coins at one time, the total amount of gold required is 220,000 gold. However, no gold will be deducted. I think it should be an issue with int32 limitations

@ddk2001
Copy link
Author

ddk2001 commented May 13, 2024

Buy-ins above 2147483647 will not be deducted for gold. This is exactly the upper limit of 32 bits

@leelf00
Copy link

leelf00 commented May 14, 2024

I push a commit due to money limit. Try last commit

@ddk2001
Copy link
Author

ddk2001 commented May 14, 2024

How did you find it. The way I came up with was to change the upper limit of the MAX_MONEY_AMOUNT

@leelf00
Copy link

leelf00 commented May 14, 2024

MAX_MONEY_AMOUNT is uint64.

@ddk2001
Copy link
Author

ddk2001 commented May 14, 2024

Yes, it is 64 bits, and the values can be large.

@leelf00
Copy link

leelf00 commented May 14, 2024

The 32bit uint32 limit is caused by code. DB column is also right.

@ddk2001
Copy link
Author

ddk2001 commented May 14, 2024

You just have to find the specific location.

@ddk2001
Copy link
Author

ddk2001 commented May 23, 2024

For advice, this version of the item upgrade is only up to level 4, and I remember that the official uniform has level 6 or 8. Do you know how to modify the location.

@leelf00
Copy link

leelf00 commented May 23, 2024

Could you find some wowhead or old wiki links for me?

@ddk2001
Copy link
Author

ddk2001 commented May 23, 2024

I have a wiki here, but I don't think it's very useful. https://warcraft.huijiwiki.com/wiki/%E9%A6%96%E9%A1%B5

@leelf00
Copy link

leelf00 commented May 23, 2024

https://wowpedia.fandom.com/wiki/Patch_5.4.8
In MOP, item can upgrade 2 times.

@ddk2001
Copy link
Author

ddk2001 commented May 23, 2024

I think this source code should have been modified. But I can't find the location where this item was upgraded, and the database hasn't found it either. I think it should be in DBC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants