diff -ur proftpd-1.2.0rc2.orig/src/log.c proftpd-1.2.0rc2/src/log.c --- proftpd-1.2.0rc2.orig/src/log.c Wed Jul 26 13:03:17 2000 +++ proftpd-1.2.0rc2/src/log.c Wed Aug 9 16:57:44 2000 @@ -529,21 +529,14 @@ int log_wtmp(char *line, char *name, char *host, p_in_addr_t *ip) { - struct stat buf; struct utmp ut; int res = 0; - static int fd = -1; #if defined(SVR4) || defined(__SVR4) -#if !(defined(LINUX) || defined(__hpux) || defined (_AIX)) +#if !(defined(__hpux) || defined (_AIX)) +#if (defined(LINUX) && defined(__GLIBC__)) /* This "auxilliary" utmp doesn't exist under linux. */ struct utmpx utx; - static int fdx = -1; - - if(fdx < 0 && (fdx = open(WTMPX_FILE, O_WRONLY | O_APPEND, 0)) < 0) { - log_pri(LOG_WARNING,"wtmpx %s: %s",WTMPX_FILE,strerror(errno)); - return -1; - } /* Unfortunately, utmp string fields are terminated by '\0' if they are * shorter than the size of the field, but if they are exactly the size of @@ -551,7 +544,6 @@ * Insane if you ask me. Unless there's massive uproar, I prefer to err on * the side of caution and always null-terminate our strings. */ - if(fstat(fdx,&buf) == 0) { memset(&utx,0,sizeof(utx)); sstrncpy(utx.ut_user,name,sizeof(utx.ut_user)); sstrncpy(utx.ut_id,"ftp",sizeof(utx.ut_user)); @@ -566,22 +558,11 @@ utx.ut_type = DEAD_PROCESS; utx.ut_exit.e_termination = 0; utx.ut_exit.e_exit = 0; - if(write(fdx,(char*)&utx,sizeof(utx)) != sizeof(utx)) - ftruncate(fdx, buf.st_size); - } else { - log_debug(DEBUG0,"%s fstat(): %s",WTMPX_FILE,strerror(errno)); - res = -1; - } + updwtmpx(_PATH_WTMP, &utx); #endif #endif /* SVR4 */ - if(fd < 0 && (fd = open(WTMP_FILE,O_WRONLY|O_APPEND,0)) < 0) { - log_pri(LOG_WARNING,"wtmp %s: %s",WTMP_FILE,strerror(errno)); - return -1; - } - - if(fstat(fd,&buf) == 0) { memset(&ut,0,sizeof(ut)); #ifdef HAVE_UTMAXTYPE #ifdef LINUX @@ -612,12 +593,7 @@ #endif /* HAVE_UT_UT_HOST */ time(&ut.ut_time); - if(write(fd,(char*)&ut,sizeof(ut)) != sizeof(ut)) - ftruncate(fd,buf.st_size); - } else { - log_debug(DEBUG0,"%s fstat(): %s",WTMP_FILE,strerror(errno)); - res = -1; - } + updwtmp(_PATH_WTMP, &utx); return res; }