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

Using the GM command will cause the server to crash #63

Closed
wthslw001 opened this issue Dec 6, 2018 · 14 comments
Closed

Using the GM command will cause the server to crash #63

wthslw001 opened this issue Dec 6, 2018 · 14 comments

Comments

@wthslw001
Copy link

Current behavior:
Using the GM command will cause the server to crash, for example: .npc
Use the latest core
Expected behavior:

Steps to reproduce:

  1. Using the GM command will cause the server to crash, for example: .npc
  2. Use the latest core

Client version:

Commit hash:

Operating system:

@ratkosrb
Copy link
Contributor

ratkosrb commented Dec 6, 2018

I can't reproduce it. It is not crashing for me.

@wthslw001
Copy link
Author

I can't reproduce it. It is not crashing for me.

You can use the command:.add npc

@wthslw001
Copy link
Author

I don't know if this is helpful for solving the problem of downtime.

@ratkosrb
Copy link
Contributor

ratkosrb commented Dec 7, 2018

There is no such command. If you mean .npc add this does not crash the server when i use it.

@wthslw001
Copy link
Author

There is no such command. If you mean .npc add this does not crash the server when i use it.

.npc

world of warcraft 2018 12 08 - 11 31 50 01 3

@wthslw001
Copy link
Author

Entering the GM command will cause the program to crash.

@wthslw001
Copy link
Author

Wrong GM command

@alexkulya
Copy link
Contributor

  • When i entering a command .npc the subcommand menu appears. Example in the screenshot:

screenshot_1

  • When entering a command .npc add I get an incorrect syntax message:

screenshot_2

  • When entering a command .npc add 'EntryID' the requested creature is added to my location.

Used last development branch revision. No crash. In game use gm level 6 access.

@wthslw001
Copy link
Author

  • When i entering a command .npc the subcommand menu appears. Example in the screenshot:

screenshot_1

  • When entering a command .npc add I get an incorrect syntax message:

screenshot_2

  • When entering a command .npc add 'EntryID' the requested creature is added to my location.

Used last development branch revision. No crash. In game use gm level 6 access.

Hello, this is the case, if you enter the wrong GM command continuously, it will cause the program to crash.

@ratkosrb
Copy link
Contributor

ratkosrb commented Dec 9, 2018

Nothing happens when i spam it.

@wthslw001
Copy link
Author

Nothing happens when i spam it.

I am not sure if it is because of the code compilation environment.
Cmake3.11 + VS 2015 X64 +TBB2017

@kotishe
Copy link

kotishe commented Dec 13, 2018

Hello! There are only two commands in the table, so it should be?
image

Levelup also have incorect syntax on debian x64

@ratkosrb
Copy link
Contributor

This table is for overriding the required rank set in Chat.cpp. You don't need to add commands there unless you want to change the minimum rank for it.

@wthslw001
Copy link
Author

此问题已解决。

spirehawk pushed a commit to spirehawk/westfall that referenced this issue Mar 21, 2022
* Followup to pet autocast

* Followup to pet autocast
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants