Re: [PATCH] Allow zsystem flock to query without second-lasting timeout

[Sebastian Gniazdowski <sgniazdowski@xxxxxxxxx>]

Really cool, thanks, I will use this via is-at-least, now I've
included cropped-down version of util-linux/flock in my plugin, it
amazingly compiles on OS X (so probably also FreeBSD), and of course
Linux

Best regards,
Sebastian Gniazdowski


On 10 September 2016 at 19:01, Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx> wrote:
> On Sep 10,  5:49pm, Sebastian Gniazdowski wrote:
> }
> } attached patch is dumb, and allows to use negative timeout, which
> } results in a quick (non-waiting) single fcntl() call.
>
> I think a better approach would be to allow a zero timeout.  (Why was
> there already a cast of time_t to time_t in the time(NULL) expression?)
>
> Hopefully no one is already using the undocumented behavior that
> "zsystem flock -t 0 ..." nevertheless waited forever.
>
>
> diff --git a/Src/Modules/system.c b/Src/Modules/system.c
> index 1ee61c0..afaec26 100644
> --- a/Src/Modules/system.c
> +++ b/Src/Modules/system.c
> @@ -521,7 +521,7 @@ static int
>  bin_zsystem_flock(char *nam, char **args, UNUSED(Options ops), UNUSED(int func))
>  {
>      int cloexec = 1, unlock = 0, readlock = 0;
> -    time_t timeout = 0;
> +    zlong timeout = -1;
>      char *fdvar = NULL;
>  #ifdef HAVE_FCNTL_H
>      struct flock lck;
> @@ -573,7 +573,7 @@ bin_zsystem_flock(char *nam, char **args, UNUSED(Options ops), UNUSED(int func))
>                 } else {
>                     optarg = *args++;
>                 }
> -               timeout = (time_t)mathevali(optarg);
> +               timeout = mathevali(optarg);
>                 break;
>
>             case 'u':
> @@ -650,7 +650,7 @@ bin_zsystem_flock(char *nam, char **args, UNUSED(Options ops), UNUSED(int func))
>             sleep(1);
>         }
>      } else {
> -       while (fcntl(flock_fd, F_SETLKW, &lck) < 0) {
> +       while (fcntl(flock_fd, timeout == 0 ? F_SETLK : F_SETLKW, &lck) < 0) {
>             if (errflag)
>                 return 1;
>             if (errno == EINTR)