diff options
| author | Theo de Raadt <deraadt@cvs.openbsd.org> | 1997-06-21 12:44:42 +0000 |
|---|---|---|
| committer | Theo de Raadt <deraadt@cvs.openbsd.org> | 1997-06-21 12:44:42 +0000 |
| commit | f244f36c6e6994996b1b89db25a6c1f8eaa16e6a (patch) | |
| tree | 1feda9c3600ea8647b6d4b99242eeab8adbac293 /libexec/ftpd/ftpd.c | |
| parent | 77531ba3936d17261f80d142f3b17d656628efce (diff) | |
attempt to realpath() log entries
Diffstat (limited to 'libexec/ftpd/ftpd.c')
| -rw-r--r-- | libexec/ftpd/ftpd.c | 37 |
1 files changed, 18 insertions, 19 deletions
diff --git a/libexec/ftpd/ftpd.c b/libexec/ftpd/ftpd.c index 35d2eebee5f..ec1356ccfb5 100644 --- a/libexec/ftpd/ftpd.c +++ b/libexec/ftpd/ftpd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ftpd.c,v 1.37 1997/06/05 09:22:41 deraadt Exp $ */ +/* $OpenBSD: ftpd.c,v 1.38 1997/06/21 12:44:41 deraadt Exp $ */ /* $NetBSD: ftpd.c,v 1.15 1995/06/03 22:46:47 mycroft Exp $ */ /* @@ -2066,38 +2066,37 @@ logxfer(name, size, start) time_t start; { char buf[400 + MAXHOSTNAMELEN*4 + MAXPATHLEN*4]; - char path[MAXPATHLEN]; + char dir[MAXPATHLEN], path[MAXPATHLEN], rpath[MAXPATHLEN]; char vremotehost[MAXHOSTNAMELEN*4], vpath[MAXPATHLEN*4]; - char *vname, *vpw; + char *vpw; time_t now; - if ((statfd >= 0) && (getcwd(path, sizeof(path)) != NULL)) { + if ((statfd >= 0) && (getcwd(dir, sizeof(dir)) != NULL)) { time(&now); - vname = (char *)malloc(strlen(name)*4+1); - if (vname == NULL) - return; vpw = (char *)malloc(strlen((guest) ? guestpw : pw->pw_name)*4+1); - if (vpw == NULL) { - free(vname); + if (vpw == NULL) return; + + snprintf(path, sizeof path, "%s/%s", dir, name); + if (realpath(path, rpath) == NULL) { + strncpy(rpath, path, sizeof rpath-1); + rpath[sizeof rpath-1] = '\0'; } + strvis(vpath, rpath, VIS_SAFE|VIS_NOSLASH); strvis(vremotehost, remotehost, VIS_SAFE|VIS_NOSLASH); - strvis(vpath, path, VIS_SAFE|VIS_NOSLASH); - strvis(vname, name, VIS_SAFE|VIS_NOSLASH); strvis(vpw, (guest) ? guestpw : pw->pw_name, VIS_SAFE|VIS_NOSLASH); snprintf(buf, sizeof(buf), - "%.24s %d %s %qd %s/%s %c %s %c %c %s ftp %d %s %s\n", - ctime(&now), now - start + (now == start), - vremotehost, size, vpath, vname, - ((type == TYPE_A) ? 'a' : 'b'), "*" /* none yet */, - 'o', ((guest) ? 'a' : 'r'), - vpw, 0 /* none yet */, - ((guest) ? "*" : pw->pw_name), dhostname); + "%.24s %d %s %qd %s %c %s %c %c %s ftp %d %s %s\n", + ctime(&now), now - start + (now == start), + vremotehost, size, vpath, + ((type == TYPE_A) ? 'a' : 'b'), "*" /* none yet */, + 'o', ((guest) ? 'a' : 'r'), + vpw, 0 /* none yet */, + ((guest) ? "*" : pw->pw_name), dhostname); write(statfd, buf, strlen(buf)); - free(vname); free(vpw); } } |