This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: mark_closed messages


"Christopher Faylor" <cgf@redhat.com> wrote:
> Of course, maybe we should just nuke the whole concept, too.
> I think the vast majority of "problems" that the
> ProtectHandle stuff has unearthed have been "false
> positives", lately.

Thanks for the response, Chris.  It was a slew of false positives
that set me off looking at this, thinking that something was wrong
with the cygserver client code.  Of course, I didn't know they
were false when I started :-)

FWIW, all the problems I saw came from calls to setclexec_pid
after a fork, and I couldn't see any easy way to fix that code.  I
tried removing NO_COPY from the relevant data structures but then,
as I mentioned, you have to handle calls to ProtectHandle before
the parent's data is copied down (and there are quite a few such
calls).  I didn't consider using the cygheap mechanism, for no
other reason than I haven't looked at that bit yet.  Anyhow, you
understand the issues better than me.

Right now, I'm happy to know that it's nothing to do with the
cygserver code, which is what I was worried about.  And I'm glad
my fiddling around was of some use.  I'll get back to the
cygserver code now.

Cheers,

// Conrad




Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]