Path: utzoo!attcan!uunet!mcvax!unido!tub!net
From: net@tub.UUCP (Oliver Laumann)
Newsgroups: comp.sources.bugs
Subject: Re: Patch (really enhancement) for `screen' V 2.0a (of 19-Oct-88)
Keywords: screen  V2.0a
Message-ID: <850@tub.UUCP>
Date: 26 May 89 12:58:52 GMT
References: <459@mahendo.Jpl.Nasa.Gov>
Reply-To: net@tub.UUCP (Oliver Laumann)
Organization: Technical University of Berlin, Germany
Lines: 17

In article <459@mahendo.Jpl.Nasa.Gov> earle@mahendo.JPL.NASA.GOV writes:
> Haven't seen many `screen' related patches lately; here's my contribution:

There seem to be several problems with this patch.  I would have sent
this as a personal reply; but one of the problems is serious.

1)  A pipe to the "w" command is opened using "popen", but I don't
    see the corresponding pclose().

2)  Screen is usually installed as set-uid with owner root (e.g.
    to be able to call chown()).  Thus the "w" command must be
    started in a child process which properly resets it's uid
    (similar to the way the "hardcopy" files are created).

Regards,
    Oliver Laumann, Technical University of Berlin, Germany.
    ...!pyramid!tub!net   or   net@TUB.BITNET