From a6e10dbd0b0650c927a2aad5b7fa810231915c9e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= Date: Wed, 17 Jun 2020 11:56:58 +0100 Subject: [PATCH] lxc: remove use of the terms 'master' and 'slave' in PTY setup MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The two sides of a PTY can be referred to as primary and secondary TTYs. Reviewed-by: Peter Krempa Signed-off-by: Daniel P. Berrangé --- src/lxc/lxc_controller.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/lxc/lxc_controller.c b/src/lxc/lxc_controller.c index 2e865e0e7c..59b44dde1f 100644 --- a/src/lxc/lxc_controller.c +++ b/src/lxc/lxc_controller.c @@ -2054,25 +2054,25 @@ static int lxcSetPersonality(virDomainDefPtr def) } /* Create a private tty using the private devpts at PTMX, returning - * the master in *TTYMASTER and the name of the slave, _from the + * the primary in @ttyprimary and the name of the secondary, _from the * perspective of the guest after remounting file systems_, in - * *TTYNAME. Heavily borrowed from glibc, but doesn't require that + * @ttyName. Heavily borrowed from glibc, but doesn't require that * devpts == "/dev/pts" */ static int -lxcCreateTty(virLXCControllerPtr ctrl, int *ttymaster, +lxcCreateTty(virLXCControllerPtr ctrl, int *ttyprimary, char **ttyName, char **ttyHostPath) { int ret = -1; int ptyno; int unlock = 0; - if ((*ttymaster = open(ctrl->devptmx, O_RDWR|O_NOCTTY|O_NONBLOCK)) < 0) + if ((*ttyprimary = open(ctrl->devptmx, O_RDWR|O_NOCTTY|O_NONBLOCK)) < 0) goto cleanup; - if (ioctl(*ttymaster, TIOCSPTLCK, &unlock) < 0) + if (ioctl(*ttyprimary, TIOCSPTLCK, &unlock) < 0) goto cleanup; - if (ioctl(*ttymaster, TIOCGPTN, &ptyno) < 0) + if (ioctl(*ttyprimary, TIOCGPTN, &ptyno) < 0) goto cleanup; /* If mount() succeeded at honoring newinstance, then the kernel @@ -2088,7 +2088,7 @@ lxcCreateTty(virLXCControllerPtr ctrl, int *ttymaster, cleanup: if (ret != 0) { - VIR_FORCE_CLOSE(*ttymaster); + VIR_FORCE_CLOSE(*ttyprimary); g_free(*ttyName); *ttyName = NULL; } -- GitLab