Re: Bash: sleep execution issue with bash loadable builtins

[Thiruvadi Rajaraman]

Hi Chester,

Thanks a lot for your review comments.

I reworked on the fix to solve bash sleep issue and here attached the patch.

>From include/bits/signum.h,

#define SIG_DFL ((__sighandler_t) 0)            /* Default action.  */

In the attached patch fix, signal(SIGCHLD, SIG_DFL), SIG_DFL performs
the default action to ignore the interrupt signal from child which
informs the kernel that there is no user signal handler for the given
signal, and default action to ignore the signal to terminate the
program.

Though SIG_IGN ignores the given signal, faced a side effect in bash sleep like
it sleep for more seconds than the actual given seconds and it affects the
behaviour in both background and active shell sleep process.

With the attached fix, bash sleep works as expected.

Please kindly review and suggest comments.

Thanks,

Thiruvadi Rajaraman

ufuncs_bash_sleep_fix.patch
Description: Text Data