Add warning messages when checking for bad fds after select() returns an error.

[meshlink] / src / net.c

diff --git a/src/net.c b/src/net.c
index 5689a0401883d9b4b3836f35dcbab09eef663eea..99c29c943290b1ea7636b7b01dd9ed072a965665 100644 (file)
--- a/src/net.c
+++ b/src/net.c
@@ -627,7 +627,7 @@ static void periodic_handler(event_loop_t *loop, void *data) {
                        n->status.dirty = false;
                }
 
-               if(n->status.validkey && n->last_req_key + 3600 < mesh->loop.now.tv_sec) {
+               if(n->status.reachable && n->status.validkey && n->last_req_key + 3600 < mesh->loop.now.tv_sec) {
                        logger(mesh, MESHLINK_DEBUG, "SPTPS key renewal for node %s", n->name);
                        devtool_sptps_renewal_probe((meshlink_node_t *)n);
 
@@ -700,7 +700,7 @@ void retry(meshlink_handle_t *mesh) {
 
                int sock = socket(sa.sa.sa_family, SOCK_STREAM, IPPROTO_TCP);
 
-               if(sock != -1) {
+               if(sock == -1) {
                        continue;
                }
 
@@ -713,9 +713,11 @@ void retry(meshlink_handle_t *mesh) {
        }
 
        /* Kick the ping timeout handler */
-       timeout_set(&mesh->loop, &mesh->pingtimer, &(struct timespec) {
-               0, 0
-       });
+       if(mesh->pingtimer.cb) {
+               timeout_set(&mesh->loop, &mesh->pingtimer, &(struct timespec) {
+                       0, 0
+               });
+       }
 }
 
 /*
@@ -733,7 +735,7 @@ void main_loop(meshlink_handle_t *mesh) {
        mesh->datafromapp.signum = 0;
        signal_add(&mesh->loop, &mesh->datafromapp, meshlink_send_from_queue, mesh, mesh->datafromapp.signum);
 
-       if(!event_loop_run(&mesh->loop, &mesh->mutex)) {
+       if(!event_loop_run(&mesh->loop, mesh)) {
                logger(mesh, MESHLINK_ERROR, "Error while waiting for input: %s", strerror(errno));
                call_error_cb(mesh, MESHLINK_ENETWORK);
        }