Call set_remains() before set_state().

set_state() may call plugin callbacks, which can destroy the action (more or
less direclty), as seen with github issue #326.

diff --git a/src/kernel/resource/Action.cpp b/src/kernel/resource/Action.cpp
index 3ea572b..cf86b5c 100644 (file)
--- a/src/kernel/resource/Action.cpp
+++ b/src/kernel/resource/Action.cpp
@@ -47,8 +47,8 @@ Action::~Action()
 void Action::finish(Action::State state)
 {
   finish_time_ = surf_get_clock();
-  set_state(state);
   set_remains(0);
+  set_state(state);
 }
 
 Action::State Action::get_state() const