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

Map-server crash when using @reloadscript if PK Mode is enabled. #2242

Open
shadow-mo0n opened this Issue Oct 2, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@shadow-mo0n

shadow-mo0n commented Oct 2, 2018

Issue Prelude

  • I have not modified the source prior to reproducing this issue.
  • I am using the latest version of Hercules.
  • I am aware that this report will be closed or deleted if it becomes obvious that I am stating the false.

Description

Map-server is crashing when using @reloadscript if PK Mode is enabled.
For GDB trace see this topic: http://herc.ws/board/topic/16348-map-server-crash-when-using-reloadscript/

Hercules rev. hash/commit:

Git revision src: 852c133
Git revision scripts: 852c133

Operating System

OS version: 'Windows 8 Professional (build 9200) [x86_64]'
CPU: 'x86_64 CPU, Family 6, Model 60, Stepping 3 [4]'
Compiled with Microsoft Visual C++ 2015 (v1900)

@dastgirp

This comment has been minimized.

Show comment
Hide comment
@dastgirp

dastgirp Oct 2, 2018

Member

@Helianthella
Probably this is causing it: 3a8705a

Member

dastgirp commented Oct 2, 2018

@Helianthella
Probably this is causing it: 3a8705a

@Helianthella Helianthella self-assigned this Oct 2, 2018

@Helianthella

This comment has been minimized.

Show comment
Hide comment
@Helianthella

Helianthella Oct 2, 2018

Contributor

@shadow-mo0n could you check if #2247 fixes the issue for you?

Contributor

Helianthella commented Oct 2, 2018

@shadow-mo0n could you check if #2247 fixes the issue for you?

@shadow-mo0n

This comment has been minimized.

Show comment
Hide comment
@shadow-mo0n

shadow-mo0n Oct 3, 2018

@Helianthella Thank you. This fix the issue.

shadow-mo0n commented Oct 3, 2018

@Helianthella Thank you. This fix the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment