* PATCH: RE: PATCH: Re: zpty and controlling tty (and other fd's)
@ 2000-05-08 11:01 Sven Wischnowsky
0 siblings, 0 replies; only message in thread
From: Sven Wischnowsky @ 2000-05-08 11:01 UTC (permalink / raw)
To: zsh-workers
Andrej Borsenkow wrote:
> > > Note fd's 3 and 13. 3 is master side of pty (child does not need it,
> > > does it?)
> >
> > Of course I tried to close it -- and on Linux this messed up the
> > parent's (i.e. the shell's) terminal settings. Don't know how this
> > came.
>
> ...
>
> I do not use Linux, but until this is cleared there - porbably under
> #ifndef __LINUX (or whatever)?
Of course I now can't reproduce it. So lets just put it in and see if
anyone complains. Maybe I did something else wrong back then...
Bye
Sven
Index: Src/Modules/zpty.c
===================================================================
RCS file: /cvsroot/zsh/zsh/Src/Modules/zpty.c,v
retrieving revision 1.6
diff -u -r1.6 zpty.c
--- Src/Modules/zpty.c 2000/05/05 09:12:50 1.6
+++ Src/Modules/zpty.c 2000/05/08 10:59:48
@@ -352,6 +352,7 @@
dup2(slave, 2);
close(slave);
+ close(master);
if (SHTTY != -1)
close(SHTTY);
--
Sven Wischnowsky wischnow@informatik.hu-berlin.de
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2000-05-08 11:02 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2000-05-08 11:01 PATCH: RE: PATCH: Re: zpty and controlling tty (and other fd's) Sven Wischnowsky
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).