mirrors/busybox
0
0
Fork 0
mirror of https://git.busybox.net/busybox.git synced 2024-11-23 05:33:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

timeout: allow fractional seconds in timeout values

Browse Source 
The 'timeout' applet uses parse_duration_str() to obtain its
timeout values.  The default configuration enables float durations.

However, the applet silently ignores fractional seconds.  This
results in unexpected behaviour:

   $ timeout 5.99 sleep 5.1; echo $?
   Terminated
   143

When float durations are enabled ensure that any fractional seconds
are taken into account.

function                                             old     new   delta
timeout_wait                                          44      92     +48
timeout_main                                         383     365     -18
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/1 up/down: 48/-18)             Total: 30 bytes

Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Ron Yorston 2024-04-08 13:12:25 +01:00 committed by Denys Vlasenko
parent b20b3790b4
commit e4b5ccd13b
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
coreutils/timeout.c
View File

@ -47,11 +47,16 @@
#include "libbb.h"
static NOINLINE int timeout_wait(int timeout, pid_t pid)
static NOINLINE int timeout_wait(duration_t timeout, pid_t pid)
{
/* Just sleep(HUGE_NUM); kill(parent) may kill wrong process! */
while (1) {
sleep1();
#if ENABLE_FLOAT_DURATION
if (timeout < 1)
sleep_for_duration(timeout);
else
#endif
sleep1();
if (--timeout <= 0)
break;
if (kill(pid, 0)) {
@ -68,8 +73,8 @@ int timeout_main(int argc UNUSED_PARAM, char **argv)
int signo;
int status;
int parent = 0;
int timeout;
int kill_timeout;
duration_t timeout;
duration_t kill_timeout;
pid_t pid;
#if !BB_MMU
char *sv1, *sv2;