Prevent meshlink_errno from being set incorrectly by meshlink_invite()

We called a public API function inside meshlink_invite() to check that we
don't try to invite a node that's already known. That causes it to set
meshlink_errno to MESHLINK_ENOENT. Fix this by calling lookup_node()
instead.

diff --git a/src/meshlink.c b/src/meshlink.c
index 11e45503c4ccbebc9a9098c89a3ae7d52c27008f..0fbcf1535379810fd2295d4d28287a87b1beb774 100644 (file)
--- a/src/meshlink.c
+++ b/src/meshlink.c
@@ -2428,7 +2428,7 @@ char *meshlink_invite_ex(meshlink_handle_t *mesh, meshlink_submesh_t *submesh, c
        }
 
        // Ensure no other nodes know about this name
        }
 
        // Ensure no other nodes know about this name

-       if(meshlink_get_node(mesh, name)) {
+       if(lookup_node(mesh, name)) {

                logger(mesh, MESHLINK_ERROR, "A node with name %s is already known!\n", name);
                meshlink_errno = MESHLINK_EEXIST;
                pthread_mutex_unlock(&mesh->mutex);
                logger(mesh, MESHLINK_ERROR, "A node with name %s is already known!\n", name);
                meshlink_errno = MESHLINK_EEXIST;
                pthread_mutex_unlock(&mesh->mutex);