patch set for #90289
Nikos Mavrogiannopoulos
nmav at redhat.com
Wed Jun 24 07:13:30 PDT 2015
On Wed, 2015-06-24 at 14:53 +0100, David Woodhouse wrote:
> On Wed, 2015-06-24 at 15:50 +0200, Nikos Mavrogiannopoulos wrote:
> > On Wed, 2015-06-24 at 14:41 +0100, David Woodhouse wrote:
> >
> > > > A better/more precise fix (replacement for 0002-Do-not-finalize
> > > > -modules-created-in-another-pid.patch) is attached.
> > > Hm, what if the parent exits but the child remains. Doesn't the
> > > parent
> > > deinitialise the functions that the child still needs? Don't you
> > > need
> > > some kind of refcount? Not that I know how you can achieve that,
> > > without locking in the atfork child handler.
> >
> > Correct, that seems like a source of many troubles. I think a ref
> > counter for that would complicate things a lot. Would it make sense
> > not
> > to release it at all? It will get released on the last user exits.
>
> Perhaps.
I think that makes sense short term, to avoid more crashes with the
current libffi. I've put my recommended patch set which fixes p11-kit
at:
http://people.redhat.com/nmavrogi/p11-kit/
It would be nice if Stef you could provide a comment and/or merge them.
> Or maybe we should just file a libffi bug and expect libffi to take
> care of the fork handling if it's going to do things this way?
https://bugzilla.redhat.com/show_bug.cgi?id=1235322
regards,Nikos
More information about the p11-glue
mailing list