fixup! Add signal functions (sigmodify, pending)

Author: Physic69

lib/lualinux.c

@@ -123,7 +123,6 @@ static int lualinux_schedule(lua_State *L)
 * @function sigmask
 * @tparam integer sig Signal number
 * @tparam[opt] integer cmd 0=BLOCK (default), 1=UNBLOCK
-* @treturn boolean `true` on success
 * @raise error string on failure (EINVAL, EPERM, etc.)
 * @within linux
 * @usage
@@ -141,8 +140,7 @@ static int lualinux_sigmask(lua_State *L)
     sigaddset(&newmask, signum);
 
     lunatik_try(L, sigprocmask, cmd, &newmask, NULL);
-    lua_pushboolean(L, true);
-    return 1;
+    return 0;
 }
 
 /***
@@ -151,62 +149,26 @@ static int lualinux_sigmask(lua_State *L)
 * @function sigpending
 * @treturn boolean
 * @within linux
+* @usage
+*   linux.sigpending()
 */
 static int lualinux_sigpending(lua_State *L)
 {
  	lua_pushboolean(L, signal_pending(current));
     return 1;
 }
 
-/***
-* Kills a process by sending a signal.
-* By default, sends SIGKILL.
-* An optional second argument can specify a different signal (either by number or by using the constants from `linux.signal`).
-*
-* @function kill
-* @tparam integer pid Process ID to kill.
-* @tparam[opt] integer sig Signal number to send (default: `linux.signal.KILL`).
-* @treturn boolean `true` if the signal was sent successfully.
-* @treturn[error] boolean `false` followed by an error number if the operation fails.
-* @raise Errors:
-*   - (3): The specified PID doesn't exist
-*   - other errno values depending on the failure cause (e.g., `EPERM`, `EINVAL`, etc.)
-* @usage
-*   linux.kill(1234)  -- Kill process 1234 with SIGKILL (default)
-*   linux.kill(1234, linux.signal.TERM)  -- Kill process 1234 with SIGTERM
-*/
-static int lualinux_kill(lua_State *L)
-{
-	pid_t nr = (pid_t)luaL_checkinteger(L, 1);
-  	int sig = luaL_optinteger(L, 2, SIGKILL);
- 	struct pid *pid = find_get_pid(nr);
-
-	int ret = ESRCH;
-	if (pid == NULL) 
-        	goto err;    
-	
-    	ret = kill_pid(pid, sig, 1);
-    	put_pid(pid);
-    
-    	if (ret) 
-		goto err;
-    
-    	lua_pushboolean(L, true);
-    	return 1;
-err:
-	lua_pushboolean(L, false);
-	lua_pushinteger(L, ret);
-	return 2;
-}
-
 /***
 * Checks signal state for current task.
 *
 * @function sigstate
 * @tparam integer sig Signal number
-* @tparam[opt] string state One of: sigstate_opts
+* @tparam[opt] string state One of: "blocked", "pending", "allowed"
 * @treturn boolean
 * @within linux
+* @usage
+*   linux.sigstate(15) -- check if SIGTERM is blocked (default)
+*   linux.sigstate(linux.signal.TERM, "pending")
 */
 static int lualinux_sigstate(lua_State *L)
 {
@@ -216,11 +178,10 @@ static int lualinux_sigstate(lua_State *L)
         SIGSTATE_ALLOWED,
     };
 
-    static const char *const sigstate_opts[] = {
+    const char *const sigstate_opts[] = {
         [SIGSTATE_BLOCKED] = "blocked",
         [SIGSTATE_PENDING] = "pending",
         [SIGSTATE_ALLOWED] = "allowed",
-        NULL
     };
 
     int signum = luaL_checkinteger(L, 1);
@@ -243,6 +204,47 @@ static int lualinux_sigstate(lua_State *L)
     return 1;
 }
 
+/***
+* Kills a process by sending a signal.
+* By default, sends SIGKILL.
+* An optional second argument can specify a different signal (either by number or by using the constants from `linux.signal`).
+*
+* @function kill
+* @tparam integer pid Process ID to kill.
+* @tparam[opt] integer sig Signal number to send (default: `linux.signal.KILL`).
+* @treturn boolean `true` if the signal was sent successfully.
+* @treturn[error] boolean `false` followed by an error number if the operation fails.
+* @raise Errors:
+*   - (3): The specified PID doesn't exist
+*   - other errno values depending on the failure cause (e.g., `EPERM`, `EINVAL`, etc.)
+* @usage
+*   linux.kill(1234)  -- Kill process 1234 with SIGKILL (default)
+*   linux.kill(1234, linux.signal.TERM)  -- Kill process 1234 with SIGTERM
+*/
+static int lualinux_kill(lua_State *L)
+{
+	pid_t nr = (pid_t)luaL_checkinteger(L, 1);
+  	int sig = luaL_optinteger(L, 2, SIGKILL);
+ 	struct pid *pid = find_get_pid(nr);
+
+	int ret = ESRCH;
+	if (pid == NULL) 
+        	goto err;    
+	
+    	ret = kill_pid(pid, sig, 1);
+    	put_pid(pid);
+    
+    	if (ret) 
+		goto err;
+    
+    	lua_pushboolean(L, true);
+    	return 1;
+err:
+	lua_pushboolean(L, false);
+	lua_pushinteger(L, ret);
+	return 2;
+}
+
 /***
 * Controls kernel tracing.
 * Turns kernel tracing on or off via `tracing_on()` and `tracing_off()`.