Re: [patch] poll.2: timeout_ts is a pointer, so use -> not . for member access

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Richard,

On 23 December 2015 at 20:30, richardvoigt@xxxxxxxxx
<richardvoigt@xxxxxxxxx> wrote:
> From the context, it is apparent that in the code explaining ppoll in
> terms of poll, timeout_ts must be a pointer.
>
> Usage #1:   ready = ppoll(&fds, nfds, timeout_ts, &sigmask);
>
> Usage #2:    (timeout_ts == NULL)
>
> Thus member access in (timeout_ts.tv_sec * 1000 + timeout_ts.tv_nsec /
> 1000000) is an error.

Thanks. Patch applied.

Cheers,

Michael


> man2/poll.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/man2/poll.2 b/man2/poll.2
> index bcbecad..34b55a6 100644
> --- a/man2/poll.2
> +++ b/man2/poll.2
> @@ -266,7 +266,7 @@ executing the following calls:
>      int timeout;
>
>      timeout = (timeout_ts == NULL) ? \-1 :
> -              (timeout_ts.tv_sec * 1000 + timeout_ts.tv_nsec / 1000000);
> +              (timeout_ts\->tv_sec * 1000 + timeout_ts\->tv_nsec / 1000000);
>      pthread_sigmask(SIG_SETMASK, &sigmask, &origmask);
>      ready = poll(&fds, nfds, timeout);
>      pthread_sigmask(SIG_SETMASK, &origmask, NULL);



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux