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

Core: Added ABORT() macro to prevent the usage of ASSERT(false) as a quick hack to crash the core misusing assert #2273

Merged
30 commits merged into from Sep 26, 2019

Conversation

Viste
Copy link
Contributor

@Viste Viste commented Sep 8, 2019

CHANGES PROPOSED:
  • add ABORT macros
Target branch(es):
  • master

Viste and others added 29 commits January 4, 2019 23:34
Signed-off-by: fossabot <badges@fossa.io>
Add license scan report and status
… quick hack to crash the core misusing assert
@Viste
Copy link
Contributor Author

Viste commented Sep 9, 2019

@milestorme dont cry baby we love you to :)))

@Viste Viste changed the title Core: Added ABORT() macro to prevent the usage of ASSERT(false) as a quick hack to crash the core misusing asser Core: Added ABORT() macro to prevent the usage of ASSERT(false) as a quick hack to crash the core misusing assert Sep 12, 2019
@Viste
Copy link
Contributor Author

Viste commented Sep 23, 2019

so what about review or something?

@ghost
Copy link

ghost commented Sep 23, 2019

I'm sorry, but I can't do this because I don't know what this is about, why we need this or what this affects. I'm not good at C++.

@Viste
Copy link
Contributor Author

Viste commented Sep 23, 2019

I'm sorry, but I can't do this because I don't know what this is about, why we need this or what this affects. I'm not good at C++.

to prevent the usage of ASSERT(false) as a quick hack to crash the core

@ghost
Copy link

ghost commented Sep 23, 2019

Yes, I read that, but I don't understand why it is bad to use ASSERT(false). You also added some changes to Errors.cpp and Errors.h that I also don't understand.

@ghost
Copy link

ghost commented Sep 23, 2019

As I told, I'm quite a noob at C++, sorry.

@Viste
Copy link
Contributor Author

Viste commented Sep 23, 2019

For better formatting(human readable) and ace replace :)

@PkllonG
Copy link
Contributor

PkllonG commented Sep 23, 2019

I didn't know how to test it, so I'll use it on my server.

@FrancescoBorzi
Copy link
Member

@pklloveyou that will be enough, thanks

@PkllonG
Copy link
Contributor

PkllonG commented Sep 24, 2019

The server was running for 24 hours with no errors found

@Winfidonarleyan
Copy link
Member

Test in win at VS 16.2. Perfect work

C:\Users\Kargatum\Desktop\D\Core\Azeroth>authserver.exe
AzerothCore rev. 24ba6797449c+ 2019-09-07 06:40:16 +0300 (ass branch) (Win64, RelWithDebInfo) (authserver)
<Ctrl-C> to stop.

   █████╗ ███████╗███████╗██████╗  ██████╗ ████████╗██╗  ██╗
  ██╔══██╗╚══███╔╝██╔════╝██╔══██╗██╔═══██╗╚══██╔══╝██║  ██║
  ███████║  ███╔╝ █████╗  ██████╔╝██║   ██║   ██║   ███████║
  ██╔══██║ ███╔╝  ██╔══╝  ██╔══██╗██║   ██║   ██║   ██╔══██║
  ██║  ██║███████╗███████╗██║  ██║╚██████╔╝   ██║   ██║  ██║
  ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝  ╚═╝ ╚═════╝    ╚═╝   ╚═╝  ╚═╝
                                ██████╗ ██████╗ ██████╗ ███████╗
                                ██╔════╝██╔═══██╗██╔══██╗██╔═══╝
                                ██║     ██║   ██║██████╔╝█████╗
                                ██║     ██║   ██║██╔══██╗██╔══╝
                                ╚██████╗╚██████╔╝██║  ██║███████╗
                                 ╚═════╝ ╚═════╝ ╚═╝  ╚═╝╚══════╝

          AzerothCore 3.3.5a  -  www.azerothcore.org

Using configuration file authserver.conf.

C:\Git\azerothcore-wotlk\src\server\authserver\Main.cpp:131 in ace_main_i ABORTED.

@ghost ghost merged commit 854b426 into azerothcore:master Sep 26, 2019
This pull request was closed.
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

Successfully merging this pull request may close these issues.

None yet

6 participants