From 1cf9f9fadf54033bfdb81b6b863e758994fea986 Mon Sep 17 00:00:00 2001 From: brian Date: Tue, 18 May 1999 01:37:52 +0000 Subject: Move the tty locking stuff back out to physical.c so that we lock the device *before* we open it. We still open the device only once - and then ask all our handlers if they know how to handle it. --- usr.sbin/ppp/ppp/physical.c | 143 ++++++++++++++++++++++++++------------ usr.sbin/ppp/ppp/tty.c | 165 +++++++++++++++----------------------------- 2 files changed, 156 insertions(+), 152 deletions(-) diff --git a/usr.sbin/ppp/ppp/physical.c b/usr.sbin/ppp/ppp/physical.c index d4c71cff1da..08074cd8040 100644 --- a/usr.sbin/ppp/ppp/physical.c +++ b/usr.sbin/ppp/ppp/physical.c @@ -16,7 +16,7 @@ * IMPLIED WARRANTIES, INCLUDING, WITHOUT LIMITATION, THE IMPLIED * WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR A PARTICULAR PURPOSE. * - * $Id: physical.c,v 1.8 1999/05/13 19:29:37 brian Exp $ + * $Id: physical.c,v 1.9 1999/05/18 01:37:51 brian Exp $ * */ @@ -256,41 +256,39 @@ physical_Offline(struct physical *p) log_Printf(LogPHASE, "%s: Disconnected!\n", p->link.name); } -static void -physical_ReallyClose(struct physical *p) +static int +physical_Lock(struct physical *p) { - int newsid; + int res; - log_Printf(LogDEBUG, "%s: Really close %d\n", p->link.name, p->fd); - if (p->fd >= 0) { - physical_StopDeviceTimer(p); - if (p->Utmp) { - ID0logout(p->name.base); - p->Utmp = 0; - } - newsid = tcgetpgrp(p->fd) == getpgrp(); - close(p->fd); - p->fd = -1; - log_SetTtyCommandMode(p->dl); - throughput_stop(&p->link.throughput); - throughput_log(&p->link.throughput, LogPHASE, p->link.name); - if (p->session_owner != (pid_t)-1) { - ID0kill(p->session_owner, SIGHUP); - p->session_owner = (pid_t)-1; - } - if (newsid) - bundle_setsid(p->dl->bundle, 0); - if (p->handler && p->handler->destroy) - (*p->handler->destroy)(p); - p->handler = NULL; + if (*p->name.full == '/' && p->type != PHYS_DIRECT && + (res = ID0uu_lock(p->name.base)) != UU_LOCK_OK) { + if (res == UU_LOCK_INUSE) + log_Printf(LogPHASE, "%s: %s is in use\n", p->link.name, p->name.full); + else + log_Printf(LogPHASE, "%s: %s is in use: uu_lock: %s\n", + p->link.name, p->name.full, uu_lockerr(res)); + return 0; } - *p->name.full = '\0'; - p->name.base = p->name.full; + + return 1; +} + +static void +physical_Unlock(struct physical *p) +{ + char fn[MAXPATHLEN]; + if (*p->name.full == '/' && p->type != PHYS_DIRECT && + ID0uu_unlock(p->name.base) == -1) + log_Printf(LogALERT, "%s: Can't uu_unlock %s\n", p->link.name, fn); } void physical_Close(struct physical *p) { + int newsid; + char fn[MAXPATHLEN]; + if (p->fd < 0) return; @@ -299,7 +297,43 @@ physical_Close(struct physical *p) if (p->handler && p->handler->cooked) (*p->handler->cooked)(p); - physical_ReallyClose(p); + physical_StopDeviceTimer(p); + if (p->Utmp) { + ID0logout(p->name.base); + p->Utmp = 0; + } + newsid = tcgetpgrp(p->fd) == getpgrp(); + close(p->fd); + p->fd = -1; + log_SetTtyCommandMode(p->dl); + + throughput_stop(&p->link.throughput); + throughput_log(&p->link.throughput, LogPHASE, p->link.name); + + if (p->session_owner != (pid_t)-1) { + ID0kill(p->session_owner, SIGHUP); + p->session_owner = (pid_t)-1; + } + + if (newsid) + bundle_setsid(p->dl->bundle, 0); + + if (*p->name.full == '/') { + snprintf(fn, sizeof fn, "%s%s.if", _PATH_VARRUN, p->name.base); +#ifndef RELEASE_CRUNCH + if (ID0unlink(fn) == -1) + log_Printf(LogALERT, "%s: Can't remove %s: %s\n", + p->link.name, fn, strerror(errno)); +#else + ID0unlink(fn); +#endif + } + physical_Unlock(p); + if (p->handler && p->handler->destroy) + (*p->handler->destroy)(p); + p->handler = NULL; + p->name.base = p->name.full; + *p->name.full = '\0'; } void @@ -780,6 +814,23 @@ physical_SetDevice(struct physical *p, const char *name) static void physical_Found(struct physical *p) { + FILE *lockfile; + char fn[MAXPATHLEN]; + + if (*p->name.full == '/') { + snprintf(fn, sizeof fn, "%s%s.if", _PATH_VARRUN, p->name.base); + lockfile = ID0fopen(fn, "w"); + if (lockfile != NULL) { + fprintf(lockfile, "%s%d\n", TUN_NAME, p->dl->bundle->unit); + fclose(lockfile); + } +#ifndef RELEASE_CRUNCH + else + log_Printf(LogALERT, "%s: Can't create %s: %s\n", + p->link.name, fn, strerror(errno)); +#endif + } + throughput_start(&p->link.throughput, "physical throughput", Enabled(p->dl->bundle, OPT_THROUGHPUT)); p->connect_count++; @@ -791,7 +842,7 @@ physical_Found(struct physical *p) int physical_Open(struct physical *p, struct bundle *bundle) { - int devno, h; + int devno, h, wasopen; char *dev; if (p->fd >= 0) @@ -814,19 +865,25 @@ physical_Open(struct physical *p, struct bundle *bundle) devno = 0; while (devno < p->cfg.ndev && p->fd < 0) { physical_SetDevice(p, dev); - - if (*p->name.full == '/') - p->fd = ID0open(p->name.full, O_RDWR | O_NONBLOCK); - - for (h = 0; h < NDEVICES && p->handler == NULL; h++) - p->handler = (*devices[h].create)(p); - - if (p->fd < 0) - log_Printf(LogWARN, "%s: Device (%s) must begin with a '/'," - " a '!' or be a host:port pair\n", p->link.name, - p->name.full); - else - physical_Found(p); + if (physical_Lock(p)) { + if (*p->name.full == '/') + p->fd = ID0open(p->name.full, O_RDWR | O_NONBLOCK); + + wasopen = p->fd >= 0; + for (h = 0; h < NDEVICES && p->handler == NULL; h++) + if ((p->handler = (*devices[h].create)(p)) == NULL && + wasopen && p->fd == -1) + break; + + if (p->fd < 0) { + if (h == NDEVICES) + log_Printf(LogWARN, "%s: Device (%s) must begin with a '/'," + " a '!' or be a host:port pair\n", p->link.name, + p->name.full); + physical_Unlock(p); + } else + physical_Found(p); + } dev += strlen(dev) + 1; devno++; } diff --git a/usr.sbin/ppp/ppp/tty.c b/usr.sbin/ppp/ppp/tty.c index 2fefcf68d8c..1eaa91855f9 100644 --- a/usr.sbin/ppp/ppp/tty.c +++ b/usr.sbin/ppp/ppp/tty.c @@ -23,7 +23,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * - * $Id: tty.c,v 1.5 1999/05/16 11:58:38 brian Exp $ + * $Id: tty.c,v 1.6 1999/05/18 01:37:51 brian Exp $ */ #include @@ -98,62 +98,6 @@ struct ttydevice { #define device2tty(d) ((d)->type == TTY_DEVICE ? (struct ttydevice *)d : NULL) -static int -tty_Lock(struct physical *p, int tunno) -{ - int res; - FILE *lockfile; - char fn[MAXPATHLEN]; - - if (*p->name.full != '/') - return 0; - - if (p->type != PHYS_DIRECT && - (res = ID0uu_lock(p->name.base)) != UU_LOCK_OK) { - if (res == UU_LOCK_INUSE) - log_Printf(LogPHASE, "%s: %s is in use\n", p->link.name, p->name.full); - else - log_Printf(LogPHASE, "%s: %s is in use: uu_lock: %s\n", - p->link.name, p->name.full, uu_lockerr(res)); - return (-1); - } - - snprintf(fn, sizeof fn, "%s%s.if", _PATH_VARRUN, p->name.base); - lockfile = ID0fopen(fn, "w"); - if (lockfile != NULL) { - fprintf(lockfile, "%s%d\n", TUN_NAME, tunno); - fclose(lockfile); - } -#ifndef RELEASE_CRUNCH - else - log_Printf(LogALERT, "%s: Can't create %s: %s\n", - p->link.name, fn, strerror(errno)); -#endif - - return 0; -} - -static void -tty_Unlock(struct physical *p) -{ - char fn[MAXPATHLEN]; - - if (*p->name.full != '/') - return; - - snprintf(fn, sizeof fn, "%s%s.if", _PATH_VARRUN, p->name.base); -#ifndef RELEASE_CRUNCH - if (ID0unlink(fn) == -1) - log_Printf(LogALERT, "%s: Can't remove %s: %s\n", - p->link.name, fn, strerror(errno)); -#else - ID0unlink(fn); -#endif - - if (p->type != PHYS_DIRECT && ID0uu_unlock(p->name.base) == -1) - log_Printf(LogALERT, "%s: Can't uu_unlock %s\n", p->link.name, fn); -} - /* * tty_Timeout() watches the DCD signal and mentions it if it's status * changes. @@ -316,7 +260,6 @@ tty_Free(struct physical *p) { struct ttydevice *dev = device2tty(p->handler); - tty_Unlock(p); free(dev); } @@ -376,22 +319,63 @@ static struct device basettydevice = { tty_OpenInfo }; -static struct device * -tty_SetupDevice(struct physical *p) +struct device * +tty_iov2device(int type, struct physical *p, struct iovec *iov, int *niov, + int maxiov) +{ + if (type == TTY_DEVICE) { + struct ttydevice *dev; + + /* It's one of ours ! Let's create the device */ + + dev = (struct ttydevice *)iov[(*niov)++].iov_base; + /* Refresh function pointers etc */ + memcpy(&dev->dev, &basettydevice, sizeof dev->dev); + + physical_SetupStack(p, PHYSICAL_NOFORCE); + if (dev->Timer.state != TIMER_STOPPED) { + dev->Timer.state = TIMER_STOPPED; + tty_StartTimer(p); + } + return &dev->dev; + } + + return NULL; +} + +struct device * +tty_Create(struct physical *p) { struct ttydevice *dev; struct termios ios; int oldflag; - if ((dev = malloc(sizeof *dev)) == NULL) + if (p->fd < 0 || !isatty(p->fd)) + /* Don't want this */ return NULL; + if (*p->name.full == '\0') { + physical_SetDevice(p, ttyname(p->fd)); + log_Printf(LogDEBUG, "%s: Input is a tty (%s)\n", + p->link.name, p->name.full); + } else + log_Printf(LogDEBUG, "%s: Opened %s\n", p->link.name, p->name.full); + + /* We're gonna return a ttydevice (unless something goes horribly wrong) */ + + if ((dev = malloc(sizeof *dev)) == NULL) { + /* Complete failure - parent doesn't continue trying to ``create'' */ + close(p->fd); + p->fd = -1; + return NULL; + } + memcpy(&dev->dev, &basettydevice, sizeof dev->dev); memset(&dev->Timer, '\0', sizeof dev->Timer); tcgetattr(p->fd, &ios); dev->ios = ios; - log_Printf(LogDEBUG, "%s: tty_SetupDevice: physical (get): fd = %d," + log_Printf(LogDEBUG, "%s: tty_Create: physical (get): fd = %d," " iflag = %lx, oflag = %lx, cflag = %lx\n", p->link.name, p->fd, (u_long)ios.c_iflag, (u_long)ios.c_oflag, (u_long)ios.c_cflag); @@ -421,9 +405,13 @@ tty_SetupDevice(struct physical *p) if (ioctl(p->fd, TIOCMGET, &dev->mbits) == -1) { if (p->type != PHYS_DIRECT) { + /* Complete failure - parent doesn't continue trying to ``create'' */ + log_Printf(LogWARN, "%s: Open: Cannot get physical status: %s\n", p->link.name, strerror(errno)); - physical_Close(p); + tty_Cooked(p); + close(p->fd); + p->fd = -1; return NULL; } else dev->mbits = TIOCM_CD; @@ -433,9 +421,13 @@ tty_SetupDevice(struct physical *p) oldflag = fcntl(p->fd, F_GETFL, 0); if (oldflag < 0) { + /* Complete failure - parent doesn't continue trying to ``create'' */ + log_Printf(LogWARN, "%s: Open: Cannot get physical flags: %s\n", p->link.name, strerror(errno)); - physical_Close(p); + tty_Cooked(p); + close(p->fd); + p->fd = -1; return NULL; } else fcntl(p->fd, F_SETFL, oldflag & ~O_NONBLOCK); @@ -444,48 +436,3 @@ tty_SetupDevice(struct physical *p) return &dev->dev; } - -struct device * -tty_iov2device(int type, struct physical *p, struct iovec *iov, int *niov, - int maxiov) -{ - if (type == TTY_DEVICE) { - struct ttydevice *dev; - - /* It's one of ours ! Let's create the device */ - - dev = (struct ttydevice *)iov[(*niov)++].iov_base; - /* Refresh function pointers etc */ - memcpy(&dev->dev, &basettydevice, sizeof dev->dev); - - physical_SetupStack(p, PHYSICAL_NOFORCE); - if (dev->Timer.state != TIMER_STOPPED) { - dev->Timer.state = TIMER_STOPPED; - tty_StartTimer(p); - } - return &dev->dev; - } - - return NULL; -} - -struct device * -tty_Create(struct physical *p) -{ - if (p->fd >= 0 && isatty(p->fd)) { - if (*p->name.full == '\0') { - log_Printf(LogDEBUG, "%s: Input is a tty\n", p->link.name); - physical_SetDevice(p, ttyname(p->fd)); - if (tty_Lock(p, p->dl->bundle->unit) == -1) { - close(p->fd); - p->fd = -1; - } else - return tty_SetupDevice(p); - } else if (tty_Lock(p, p->dl->bundle->unit) != -1) { - log_Printf(LogDEBUG, "%s: Opened %s\n", p->link.name, p->name.full); - return tty_SetupDevice(p); - } - } - - return NULL; -} -- cgit v1.2.3