Fixing a security hole in pinfo.
Thu Sep 11 14:08:00 GMT 2003
On Thu, Sep 11, 2003 at 12:05:42AM -0400, Pierre A. Humblet wrote:
>The flag PID_MAP_RW is added in the few pinfo constructors
>that need to be write into _pinfo if it exists.
>diff -u -p -r1.166 exceptions.cc
>--- exceptions.cc 10 Sep 2003 17:26:12 -0000 1.166
>+++ exceptions.cc 11 Sep 2003 03:40:57 -0000
>@@ -610,7 +610,7 @@ sig_handle_tty_stop (int sig)
> its list of subprocesses. */
> if (my_parent_is_alive ())
>- pinfo parent (myself->ppid);
>+ pinfo parent (myself->ppid, PID_MAP_RW);
> if (NOTSTATE (parent, PID_NOCLDSTOP))
> sig_send (parent, SIGCHLD);
The above won't need to be RW when I check in my new signal changes.
(Not that there won't be other inheritance type problems)
I'm going to hold off on checking this in until 1.5.4 is released.
Otherwise, it looks ok.
More information about the Cygwin-patches