mockingbirdnest / Principia

𝑛-Body and Extended Body Gravitation for Kerbal Space Program
MIT License
746 stars 67 forks source link

Maccollo 4.1 #1022

Closed eggrobin closed 8 years ago

eggrobin commented 8 years ago

5 instances of the following CHECK failure with the following stack trace, reported by maccollo.

flight_plan.cpp:59] Check failed: index < number_of_manœuvres() (0 vs. 0)

ksp_plugin/flight_plan.cpp:59 ksp_plugin/interface_flight_plan.cpp:244

pleroy commented 8 years ago

The theory is that the C# calls Plugin::ForgetAllHistoriesBefore which ends up calling FlightPlan::ForgetBefore which may remove some burns (the oldest ones). However, FlightPlanner is never informed of the changes, so its burn_editors_ may be left inconsistent with the state of FlightPlan.