Don't allow meshlink_join() when the storage policy is DISABLED.

It does not make sense to allow this, since we need to write host config
files during a join, otherwise MeshLink's directory would be left in an
invalid state.

diff --git a/src/meshlink.c b/src/meshlink.c
index da7305515bec02fb4687b91afa0d08b02819dd96..8b9e195fa7df2b9a96d4c949d721867d10271783 100644 (file)
--- a/src/meshlink.c
+++ b/src/meshlink.c
@@ -3025,6 +3025,11 @@ bool meshlink_join(meshlink_handle_t *mesh, const char *invitation) {
                return false;
        }
 
+       if(mesh->storage_policy == MESHLINK_STORAGE_DISABLED) {
+               meshlink_errno = MESHLINK_EINVAL;
+               return false;
+       }
+
        join_state_t state = {
                .mesh = mesh,
                .sock = -1,

diff --git a/src/meshlink.h b/src/meshlink.h
index 3be94710309da0a13c36b231578456156279b8fc..7960d31051426dc927bc53aaf308a1b272916895 100644 (file)
--- a/src/meshlink.h
+++ b/src/meshlink.h
@@ -1106,6 +1106,7 @@ char *meshlink_invite(struct meshlink_handle *mesh, struct meshlink_submesh *sub
  *  After a successfully accepted invitation, the name of the local node may have changed.
  *
  *  This function may only be called on a mesh that has not been started yet and which is not already part of an existing mesh.
+ *  It is not valid to call this function when the storage policy set to MESHLINK_STORAGE_DISABLED.
  *
  *  This function is blocking. It can take several seconds before it returns.
  *  There is no guarantee it will perform a successful join.