Re: [PATCH v2] Cygwin: Fix the address of myself

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

On Jul 24 19:11, Ken Brown wrote:
> On 7/24/2019 12:54 PM, Corinna Vinschen wrote:
> > From: Corinna Vinschen <corinna@vinschen.de>
> > 
> > v2: rephrase commit message
> > 
> > Introducing an independent Cygwin PID introduced a regression:
> > 
> > The expectation is that the myself pinfo pointer always points to a
> > specific address right in front of the loaded Cygwin DLL.
> > 
> > However, the independent Cygwin PID changes broke this.  To create
> > myself at the right address requires to call init with h0 set to
> > INVALID_HANDLE_VALUE or an existing address:
> > 
> > void
> > pinfo::init (pid_t n, DWORD flag, HANDLE h0)
> > {
> >    [...]
> >    if (!h0 || myself.h)
> >      [...]
> >    else
> >      {
> >        shloc = SH_MYSELF;
> >        if (h0 == INVALID_HANDLE_VALUE)       <-- !!!
> >          h0 = NULL;
> >      }
> > 
> > The aforementioned commits changed that so h0 was always NULL, this way
> > creating myself at an arbitrary address.
> > 
> > This patch makes sure to set the handle to INVALID_HANDLE_VALUE again
> > when creating a new process, so init knows that myself has to be created
> > in the right spot.  While at it, fix a potential uninitialized handle
> > value in child_info_spawn::handle_spawn.
> > 
> > Fixes: b5e1003722cb ("Cygwin: processes: use dedicated Cygwin PID rather than Windows PID")
> > Fixes: 88605243a19b ("Cygwin: fix child getting another pid after spawnve")
> > Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
> > ---
> >   winsup/cygwin/dcrt0.cc | 2 +-
> >   winsup/cygwin/pinfo.cc | 3 +--
> >   2 files changed, 2 insertions(+), 3 deletions(-)
> > [...]
> 
> I'll be glad to take a close look at this as you asked.  But I'm not familiar 
> with this part of the code, so it will take me a little time.
> 
> Ken

Thanks!  I accidentally pushed the patch a few minutes ago when I
was actually just planning to push the ndbm.h patch.  Anyway, I
took the opportunity to create new snapshots with all patches from
yesterday and today, so the getpgrp problems in GDB 8.1.1 and 8.2.1
should both be fixed there as well.

I'd still be glad if the two of you could check if my patch makes
sense as is.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

Attachment: signature.asc
Description: PGP signature