Another place where we can check before calling close() on an fd.

proftpd · Oct 19, 2014 · a044698 · a044698
1 parent 610542e
commit a044698
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/modules/mod_ctrls.c b/modules/mod_ctrls.c
@@ -1217,8 +1217,10 @@ static void ctrls_restart_ev(const void *event_data, void *user_data) {
     pr_ctrls_cl_t *cl = NULL;
 
     for (cl = cl_list; cl; cl = cl->cl_next) {
-      close(cl->cl_fd);
-      cl->cl_fd = -1;
+      if (cl->cl_fd >= 0) {
+        (void) close(cl->cl_fd);
+        cl->cl_fd = -1;
+      }
     }
   }