From 99e924357eaf5a3f63028d741071370e22746e9c Mon Sep 17 00:00:00 2001 From: Damien Miller Date: Sun, 18 Feb 2001 12:49:35 +1100 Subject: - (djm) Use ttyname() to determine name of tty returned by openpty() rather then risking overflow. Patch from Marek Michalkiewicz --- pty.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'pty.c') diff --git a/pty.c b/pty.c index e71bceb80..4b9370ce5 100644 --- a/pty.c +++ b/pty.c @@ -49,15 +49,19 @@ pty_allocate(int *ptyfd, int *ttyfd, char *namebuf, int namebuflen) { #if defined(HAVE_OPENPTY) || defined(BSD4_4) /* openpty(3) exists in OSF/1 and some other os'es */ - char buf[64]; + char *name; int i; - i = openpty(ptyfd, ttyfd, buf, NULL, NULL); + i = openpty(ptyfd, ttyfd, NULL, NULL, NULL); if (i < 0) { error("openpty: %.100s", strerror(errno)); return 0; } - strlcpy(namebuf, buf, namebuflen); /* possible truncation */ + name = ttyname(*ttyfd); + if (!name) + fatal("openpty returns device for which ttyname fails."); + + strlcpy(namebuf, name, namebuflen); /* possible truncation */ return 1; #else /* HAVE_OPENPTY */ #ifdef HAVE__GETPTY -- cgit v1.2.3