On Fri, Mar 29, 2019 at 5:01 AM William A Rowe Jr <wr...@rowe-clan.net> wrote:
>
> Opinions?

Anyway, our usage of readdir_r() is no more thread-safe than with the
non-_r() version, because of the (struct dirent *)thedir->entry we
pass to it (should be either on the stack or allocated for each
apr_dir_read() call), or more generally because of our design of
apr_dir_read() which does _not_ allow to call it concurrently with the
same apr_dir_t (thus pool), like almost all of our designs.

So readdir() vs readdir_r () is a non-issue to begin with IMHO (we use
readdir_r() probably like readdir() is implemented), let's just
silence the warning like in r1856274...

Reply via email to