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

Level Editor: crash when viewing, modifying, or removing particles from a level #510

Closed
maxteufel opened this issue Jul 28, 2016 · 4 comments · Fixed by #561
Closed

Level Editor: crash when viewing, modifying, or removing particles from a level #510

maxteufel opened this issue Jul 28, 2016 · 4 comments · Fixed by #561

Comments

@maxteufel
Copy link
Member

SuperTux version: git master as of 2016-07-28 13:34 UTC+2
System information: Arch Linux, Linux 4.6.4, Clang 3.8

Expected behavior

SuperTux doesn't crash.

Actual behavior

SuperTux crashes.

Steps to reproduce actual behavior
  1. Place a level using rain particles into the override-data directory recently introduced (the one where menu.stl is placed in). Example: ${buildroot}/data/levels/world2/Going_Underground.stl.
  2. Open level in level editor.
  3. Remove rain particles.
Additional debugging information

https://gist.github.com/maxteufel/b52115034c5b8827058124d2d5c031e3

@tobbi
Copy link
Member

tobbi commented Jul 28, 2016

Please test the fix.

@maxteufel
Copy link
Member Author

Still seems to be crashing in LayerIcon::get_zpos(), this time when removing/editing cloud particles: https://gist.github.com/maxteufel/dc7feed40866779e5a48d6071794175e

@maxteufel maxteufel reopened this Jul 28, 2016
@Hume2
Copy link
Member

Hume2 commented Jul 28, 2016

I can't reproduce it.

@maxteufel maxteufel changed the title Level Editor: crash when removing rain particles from a level Level Editor: crash when viewing, modifying, or removing particles from a level Jul 28, 2016
@maxteufel
Copy link
Member Author

@Hume2: it doesn't seem to be reproducible always. And it also happens with other particles (e.g. clouds).

Possibly this is related to the layer management in the editor. @tobbi suggested on our Slack that the layer could be deleted, but for some reason would still be set to a non-null value, which might cause LayerIcon::get_zpos() to crash.

tobbi added a commit to tobbi/supertux that referenced this issue Aug 20, 2016
tobbi added a commit to tobbi/supertux that referenced this issue Aug 21, 2016
Fixes SuperTux#510

Reported-by: Max Teufel <[email protected]>
Tested-by: Max Teufel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants