search

ChristopherRabotin / GMAT

unmaintained fork of NASA General Mission Analysis Tool, https://gmat.gsfc.nasa.gov/. If you encounter any issue, please contact them directly
Apache License 2.0
37 stars 15 forks source link

Segfault when deleting last planet in list #5

Closed bryan-lunt closed 3 months ago

bryan-lunt commented 8 months ago

These lines will create a segfault. It might work better to reverse iterate for deletion.

https://github.com/ChristopherRabotin/GMAT/blob/829b7c2c3c7ea73d759c338e7051f92f4f2f6f43/src/base/solarsys/SolarSystem.cpp#L2216-L2221

ChristopherRabotin commented 8 months ago

Hi Bryan,

Thanks for the info. However, this is fork of GMAT is not maintained at all. It only exists to allow for quickly searching through the GMAT code. I'm not a maintainer of GMAT, but I write an open-source alternative in Python and in Rust.

Hence, best would be to reproduce the bug and reach out to Steve Hughes and his team on Source forge.

Thanks.

On Sun, Jan 7, 2024, 22:23 Bryan Lunt @.***> wrote:

These lines will create a segfault. It might work better to reverse iterate for deletion.

https://github.com/ChristopherRabotin/GMAT/blob/829b7c2c3c7ea73d759c338e7051f92f4f2f6f43/src/base/solarsys/SolarSystem.cpp#L2216-L2221

— Reply to this email directly, view it on GitHub https://github.com/ChristopherRabotin/GMAT/issues/5, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABEZV2G4CG2TOQGHJA5SNR3YNN7GBAVCNFSM6AAAAABBQ44R3OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3DSNRZHAYTGNY . You are receiving this because you are subscribed to this thread.Message ID: @.***>