From c9e1fd9567071050e673552088265963fbca9930 Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Fri, 26 Oct 2001 17:15:17 +0000 Subject: 16176: Src/Modules/tcp.c, Src/Modules/zftp.c: after fdclose(), TCP control fd is no longer valid, but we still need to remove the session cleanly. --- Src/Modules/tcp.c | 14 ++++++++------ Src/Modules/zftp.c | 7 +++++++ 2 files changed, 15 insertions(+), 6 deletions(-) (limited to 'Src') diff --git a/Src/Modules/tcp.c b/Src/Modules/tcp.c index 1fe813e3f..9109ca7c1 100644 --- a/Src/Modules/tcp.c +++ b/Src/Modules/tcp.c @@ -291,19 +291,21 @@ tcp_close(Tcp_session sess) { int err; - if (sess && sess->fd != -1) + if (sess) { - err = close(sess->fd); - if (err) + if (sess->fd != -1) { - zwarn("connection close failed: %e", NULL, errno); - return -1; + err = close(sess->fd); + if (err) + { + zwarn("connection close failed: %e", NULL, errno); + return -1; + } } zts_delete(sess); return 0; } - zts_delete(sess); return -1; } diff --git a/Src/Modules/zftp.c b/Src/Modules/zftp.c index 8dda5f1a8..9408c7a1f 100644 --- a/Src/Modules/zftp.c +++ b/Src/Modules/zftp.c @@ -2679,6 +2679,13 @@ zfclose(int leaveparams) } if (zfsess->cin) { fclose(zfsess->cin); + /* + * We fdopen'd the TCP control fd; since we can't fdclose it, + * we need to perform a full fclose, which invalidates the + * TCP fd. + */ + if (fileno(zfsess->cin) == zfsess->control->fd) + zfsess->control->fd = -1; zfsess->cin = NULL; } if (zfsess->control) { -- cgit 1.4.1