Re: [PATCH] manual: Document the dlinfo function

[Andreas Schwab <schwab@linux-m68k.org>]

On Apr 14 2022, Florian Weimer via Libc-alpha wrote:

> +@item RTLD_DI_SERINFO
> +@item RTLD_DI_SERINFOSIZE

The second @item needs to be @itemx.

> +These requests can be used to obtain search path information for
> +@var{handle}.  For both requests, @var{arg} must point to a
> +@code{Dl_serinfo} object.  The @code{RTLD_DI_SERINFOSIZE} request must
> +be made first; it updates the @code{@var{arg}->dls_size} and
> +@code{@var{arg}->dls_cnt} members.

This is misleading since arg is a void pointer.  Perhaps just refer to
the members by their name:

  ... it updates the @code{dls_size} and @code{dls_cnt} members of the
  @code{Dl_serinfo} object.

> +This interface is prone to buffer overflows in multi-threaded because

+processes

> +@item RTLD_DI_TLS_DATA
> +This request writes the address of the TLS block (in the current thread)
> +for the shared object identified by @var{handle} to a @code{void *}
> +object at address @var{arg} (that is, @var{arg} must be of type
> +@code{void **}).

This description is weirdly different from the others wrt. how @var{arg}
is described.  I think how the other places describe it is better.

> +The reminder of this section documents the @code{_dl_find_object}

remainder

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."