From d10cd71d3758e56ed7d59e42ec9392fed2573937 Mon Sep 17 00:00:00 2001 From: Mattias Andrée Date: Fri, 22 Nov 2013 07:57:30 +0100 Subject: fix chmod and chown of tty device files MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Mattias Andrée --- src/cerberus.c | 12 ++++++++++++ src/cerberus.h | 1 + src/security.c | 4 ++-- 3 files changed, 15 insertions(+), 2 deletions(-) diff --git a/src/cerberus.c b/src/cerberus.c index f20e600..e52a48f 100644 --- a/src/cerberus.c +++ b/src/cerberus.c @@ -53,6 +53,8 @@ char* passphrase = NULL; */ int main(int argc, char** argv) { + char* tty_device = ttyname(STDIN_FILENO); + do_login(argc, argv); /* Ignore signals */ @@ -63,6 +65,16 @@ int main(int argc, char** argv) while ((wait(NULL) == -1) && (errno == EINTR)) ; + /* Regain access to the terminal */ + if (tty_device) + { + int fd = open(tty_device, O_RDWR | O_NONBLOCK); + if (fd) + dup2(fd, 0); + dup2(fd, 1); + dup2(fd, 2); + } + /* Reset terminal ownership and mode */ chown_tty(0, tty_group, 0); diff --git a/src/cerberus.h b/src/cerberus.h index 5851524..69df492 100644 --- a/src/cerberus.h +++ b/src/cerberus.h @@ -28,6 +28,7 @@ #include #include #include +#include #include #include #include diff --git a/src/security.c b/src/security.c index 3abd7b4..eb4bd3c 100644 --- a/src/security.c +++ b/src/security.c @@ -132,8 +132,8 @@ void chown_tty(uid_t owner, gid_t group, char with_fail) vcs -= VCS_LEN; vcsa -= VCSA_LEN; - strcpy(vcs, VCS); - strcpy(vcsa, VCSA); + strncpy(vcs, VCS, VCS_LEN); + strncpy(vcsa, VCSA, VCSA_LEN); #ifdef OWN_VCS if (chown(vcs, owner, group) && with_fail) fail("chown"); -- cgit v1.2.3-70-g09d2