[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Race condition in APR_DECLARE_LATE_DLL_FUNC() implementation

From: William A. Rowe Jr. <wrowe_at_rowe-clan.net>
Date: Fri, 6 Dec 2013 11:24:20 -0600

On Fri, 6 Dec 2013 16:44:52 +0100
Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com> wrote:

> On Fri, Dec 6, 2013 at 6:05 AM, William A. Rowe Jr.
> <wrowe_at_rowe-clan.net>wrote:
>
> > On Thu, 5 Dec 2013 15:01:05 +0100
> > "Bert Huijben" <bert_at_qqmail.nl> wrote:
> >
> > > I think the dll load function should be converted to a more stable
> > > pattern, that properly handles multiple threads. And perhaps we
> > > should just assume a few more NT functions to be alsways there
> > > instead of loading them dynamically.
> >
> > This is possible with the 'mandatory' call to apr_init, but I think
> > the existing pattern should persist for those who don't like to
> > call the initialization logic.
> >
>
> We currently call apr_initialize() before spawning threads or doing
> anything other APR. What else do we need to become thread-safe
> under Windows?

Working on a patch. Simply, we just need an _initialize hack that
triggers this lookup for each internally required (or not-present) fn.
Received on 2013-12-06 18:25:11 CET

This is an archived mail posted to the Subversion Dev mailing list.