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

Crash when observer is on the spaceship #3115

Closed
alex-w opened this issue Mar 20, 2023 · 1 comment
Closed

Crash when observer is on the spaceship #3115

alex-w opened this issue Mar 20, 2023 · 1 comment
Assignees
Labels
bug Something likely wrong in the code importance: critical Real showstopper, program fails here qt Issues, related to Qt framework
Milestone

Comments

@alex-w
Copy link
Member

alex-w commented Mar 20, 2023

Expected Behaviour

No crash

Actual Behaviour

Crash when observer is on the spaceship in Qt5-based builds

Steps to reproduce

  1. Open Stellarium
  2. Select planet
  3. Press Ctrl+G (flight to the selected planet)
  4. Crash

Notes

I can reproduce the issue on macOS/Windows in any Qt5-based builds when RTS data is shower, but not in Qt6-based builds. The issue was introduced in commit 3364be7

In crash dump I see:

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   stellarium                    	       0x104bd3f3c InfoPanel::setTextFromObjects(QList<QSharedPointerNoDelete<StelObject> > const&) + 244
1   stellarium                    	       0x104bd3f2c InfoPanel::setTextFromObjects(QList<QSharedPointerNoDelete<StelObject> > const&) + 228
2   stellarium                    	       0x104bd1390 StelGui::update() + 5236

and no issues in the log.txt

@alex-w alex-w added bug Something likely wrong in the code importance: critical Real showstopper, program fails here qt Issues, related to Qt framework labels Mar 20, 2023
@alex-w alex-w added this to the 23.1 milestone Mar 20, 2023
@gzotti
Copy link
Member

gzotti commented Mar 21, 2023

Hmm. Qt6 debug build. RTS computed for the moon. Select Jupiter, Ctrl-G. Crash with QList::at index out of range, pointing to somewhere in qlist.h, L423. Not really helpful...

@gzotti gzotti mentioned this issue Mar 21, 2023
13 tasks
@alex-w alex-w self-assigned this Mar 21, 2023
@alex-w alex-w closed this as completed in 9ceb240 Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something likely wrong in the code importance: critical Real showstopper, program fails here qt Issues, related to Qt framework
Development

No branches or pull requests

2 participants