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

Segmentation fault (core dumped) when overwriting new map #1734

Closed
Ev1lbl0w opened this Issue Sep 7, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@Ev1lbl0w

Ev1lbl0w commented Sep 7, 2017

I get consistent exceptions of "Segmentation fault (core dumped)" (memory leak) by simply creating a new map, saving it, and creating a new map with the exact same name, which ask me to overwrite it.

Steps to reproduce:

  • Create a New Map (any definitions) and save it.
  • Create another New Map, and save it to the same location and with the same name as the one before. (It asks to overwrite the old map)
  • After a second or two, it crashes.

I tested this on both the release (Tiled-1.0.3-x86_64) and snapshot (Tiled-2017.08.29-x86_64)

My operating system is Ubuntu 16.04 LTS 64-bits

@bjorn bjorn added the bug label Sep 7, 2017

@bjorn

This comment has been minimized.

Show comment
Hide comment
@bjorn

bjorn Sep 7, 2017

Owner

Why would you do such a thing? :-/

But yeah, it's easily reproducible and should be fixed. I'll look into fixing it soon, but I may not get around to it until next week. Thanks for the report!

Owner

bjorn commented Sep 7, 2017

Why would you do such a thing? :-/

But yeah, it's easily reproducible and should be fixed. I'll look into fixing it soon, but I may not get around to it until next week. Thanks for the report!

@bjorn

This comment has been minimized.

Show comment
Hide comment
@bjorn

bjorn Sep 12, 2017

Owner

So, the easiest way to reproduce this was actually:

  • Create a New Map
  • Press Ctrl+R (Reload)

See explanation in the commit message. :-)

Owner

bjorn commented Sep 12, 2017

So, the easiest way to reproduce this was actually:

  • Create a New Map
  • Press Ctrl+R (Reload)

See explanation in the commit message. :-)

@bjorn bjorn closed this in f593b7b Sep 12, 2017

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