Ubuntu - Zero - Crash when loading scripts

wtfcolt · July 27, 2014

Just wanted to note after the recent revisions, this is still an issue for me. I was hoping some of those ubuntu changes would help things, but no luck.

antz · July 27, 2014

Hi,
I was having serious issues on Ubuntu getting Zero to build, this update allowed me to build :D

wtfcolt · July 27, 2014

Are you having this same crash?

wtfcolt · July 28, 2014

Is there anything I can do to provide more information on this issue?

lfxgroove · July 30, 2014

Hi there, would you be able to recompile with debugging info turned on and give it a go with valgrind as well please? To compile with debug info you would do something like:
[code]
cmake .. -DCMAKE_INSTALL_PREFIX=/la/la/bin -DDEBUG=1
[/code]

And to run it with valgrind you would do something like:
[code]
valgrind /path/to/mangosd
[/code]

Please copy the output that valgrind gives you as well as a normal run with debugging enabled to this thread and i should have something more to go on. Thanks!

lfxgroove · July 30, 2014

Just noticed that this seems to be related to another bug mentioned in [url]https://www.getmangos.eu/help-amp-support/9957-zero-rel19-top-5-major-bugs.html[/url].

Would you like to try with the following patch?
[code]
From 36a585caeca6c4f3080ea2f5bd6776a0e3170e0e Mon Sep 17 00:00:00 2001
From: lfxgroove
Date: Tue, 29 Jul 2014 12:47:06 +0000
Subject: [PATCH 1/2] [Core/Realmd] Fix crash when database version doesn't
match.

The general gut feeling is that the Runnable shouldn't ref-count itself, but rather eg.
ACE_Strong_Bound_Ptr should be used for that purpose to ensure deletion at the correct
time. Comments?

Also remove #define MANGOS as that's defined on the command line when building
---
src/game/SharedDefines.h | 1 -
src/shared/Threading.cpp | 4 +++-
2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/game/SharedDefines.h b/src/game/SharedDefines.h
index 7261229..b58cafc 100644
--- a/src/game/SharedDefines.h
+++ b/src/game/SharedDefines.h
@@ -28,7 +28,6 @@
#include "Platform/Define.h"
#include

-#define MANGOS
#define CLASSIC

enum Gender
diff --git a/src/shared/Threading.cpp b/src/shared/Threading.cpp
index 668aa43..bb7e21d 100644
--- a/src/shared/Threading.cpp
+++ b/src/shared/Threading.cpp
@@ -191,7 +191,9 @@ void Thread::resume()

ACE_THR_FUNC_RETURN Thread::ThreadTask(void* param)
{
- Runnable* _task = (Runnable*)param;
+ Runnable* _task = static_cast(param);
+ _task->incReference();
+
_task->run();

// task execution complete, free referecne added at
--
2.0.1

From 6d8d16c7b375708770949065c0f0d37d57196da4 Mon Sep 17 00:00:00 2001
From: lfxgroove
Date: Tue, 29 Jul 2014 12:58:48 +0000
Subject: [PATCH 2/2] [Core] Revert part of last commit

---
src/game/SharedDefines.h | 3 +++
1 file changed, 3 insertions(+)

diff --git a/src/game/SharedDefines.h b/src/game/SharedDefines.h
index b58cafc..17fae55 100644
--- a/src/game/SharedDefines.h
+++ b/src/game/SharedDefines.h
@@ -28,6 +28,9 @@
#include "Platform/Define.h"
#include

+#ifndef MANGOS
+#define MANGOS
+#endif /* MANGOS */
#define CLASSIC

enum Gender
--
2.0.1
[/code]

You can apply it with git apply or git am. See [url]https://ariejan.net/2009/10/26/how-to-create-and-apply-a-patch-with-git/[/url] for some more info. Please reply back if it works.

wtfcolt · July 30, 2014

Compiling with patch, if it still fails, ill return will valgrind and debug info

wtfcolt · July 30, 2014

Mangos zero built 100% with patch, server loaded up with no issues. I logged in, took a quest, and killed one guy.

Think you got it, champ =)

lfxgroove · July 31, 2014

Glad to hear it! This patch is now in Rel20 as well. Thanks for testing it!

madmax · July 31, 2014

Marking this as fixed.

[url]https://github.com/mangoszero/server/commit/f1edf257b71288dc8fccba78a40ae58946bcb437[/url]

Sign In

Bug Feed

Categories

Ubuntu - Zero - Crash when loading scripts

User Feedback

Recommended Comments

wtfcolt 0

Link to comment

Share on other sites

antz 142

Link to comment

Share on other sites

wtfcolt 0

Link to comment

Share on other sites

wtfcolt 0

Link to comment

Share on other sites

lfxgroove 0

Link to comment

Share on other sites

lfxgroove 0

Link to comment

Share on other sites

wtfcolt 0

Link to comment

Share on other sites

wtfcolt 0

Link to comment

Share on other sites

lfxgroove 0

Link to comment

Share on other sites

madmax 130

Link to comment

Share on other sites

Create an account or sign in to comment

Create an account

Sign in

Recently Browsing 0 members

Contact Us

Repositories

Support

Wiki

Browse

Activity

My Activity Streams

Important Information