Don't clear node dirty flag in meshlink_stop().

It's node_write_config() itself that should unset the dirty flag, depending
on the storage policy and if storage succeeded. Otherwise, we risk that
setting the storage policy back to ENABLED after calling meshlink_stop()
will not cause pending updates to be written out by meshlink_close().

diff --git a/src/meshlink.c b/src/meshlink.c
index 26bcada773151a382772e2117e05d5bb101ce67f..da7305515bec02fb4687b91afa0d08b02819dd96 100644 (file)
--- a/src/meshlink.c
+++ b/src/meshlink.c
@@ -1846,7 +1846,9 @@ void meshlink_stop(meshlink_handle_t *mesh) {
        if(mesh->nodes) {
                for splay_each(node_t, n, mesh->nodes) {
                        if(n->status.dirty) {
-                               n->status.dirty = !node_write_config(mesh, n, false);
+                               if(!node_write_config(mesh, n, false)) {
+                                       // ignore
+                               }
                        }
                }
        }