ci: add missing check target to CI build loops

.github/workflows/CI.yml

@@ -160,6 +160,7 @@ jobs:
                 -DCMAKE_BUILD_TYPE=Release \
                 -B build
             cmake --build build
+            cmake --build build --target check
           done
           rm -rf build
 
@@ -234,6 +235,7 @@ jobs:
                   -DCMAKE_BUILD_TYPE=Release \
                   -B build
               cmake --build build
+              cmake --build build --target check
             done
             rm -rf build
 

src/limit_process.c

@@ -187,18 +187,17 @@ static void send_signal_to_processes(struct process_group *proc_group, int sig,
             continue;
         }
         pid = ((const struct process *)node->data)->pid;
-        do {
-            kill_result = kill(pid, sig);
-        } while (kill_result != 0 && errno == EINTR);
+        kill_result = kill(pid, sig);
 
         if (kill_result != 0) {
             /*
              * Signal delivery failed. Common reasons:
              * - ESRCH: Process no longer exists
              * - EPERM: Permission denied (rare in this context)
              *
-             * EINTR is transient and retried above, so all failures here
-             * indicate a process that can no longer be reliably controlled.
+             * kill() is non-blocking and cannot fail with EINTR, so any
+             * failure here indicates a process that can no longer be
+             * reliably controlled.
              * Save errno before any other calls that may clobber it.
              */
             int saved_errno = errno;

src/limiter.c

@@ -98,7 +98,6 @@
  */
 static int is_script_inaccessible_interpreter(const char *path) {
     int fd;
-    int access_result;
     int saved_errno;
     char buf[256];
     ssize_t n;
@@ -147,10 +146,7 @@ static int is_script_inaccessible_interpreter(const char *path) {
     }
 
     /* Interpreter path is inaccessible -> report 126 */
-    do {
-        access_result = access(p, F_OK);
-    } while (access_result != 0 && errno == EINTR);
-    return access_result != 0;
+    return access(p, F_OK) != 0;
 }
 
 /**

src/list.c

@@ -246,8 +246,7 @@ void clear_list(struct list *lst) {
         free(current_node);
     }
     /* Reset list to empty state */
-    lst->first = lst->last = NULL;
-    lst->count = 0;
+    init_list(lst);
 }
 
 /**
@@ -273,6 +272,5 @@ void destroy_list(struct list *lst) {
         free(current_node);
     }
     /* Reset list to empty state */
-    lst->first = lst->last = NULL;
-    lst->count = 0;
+    init_list(lst);
 }

src/process_table.c

@@ -256,7 +256,7 @@ void destroy_process_table(struct process_table *proc_table) {
         }
     }
     /* Free the bucket array itself */
-    free((void *)proc_table->buckets);
+    free(proc_table->buckets);
     proc_table->buckets = NULL;
     proc_table->hash_size = 0;
 }