diff options
| author | Peter Stephenson <pws@users.sourceforge.net> | 2002-05-15 20:38:17 +0000 |
|---|---|---|
| committer | Peter Stephenson <pws@users.sourceforge.net> | 2002-05-15 20:38:17 +0000 |
| commit | bbefa484f47ba659afa78b6a3c8621a17ed29e03 (patch) | |
| tree | 57af6127bcca84a1b2970c01f335be0ba0457d25 /Src/Modules/zftp.c | |
| parent | follow-up to 17135, backslash-quoted backslashes in double quotes weren't rig... (diff) | |
| download | zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.tar zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.tar.gz zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.tar.bz2 zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.tar.lz zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.tar.xz zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.tar.zst zsh-bbefa484f47ba659afa78b6a3c8621a17ed29e03.zip | |
17166: Src/Modules/zftp.c: check fileno of zftp control
connection before closing, not after
Diffstat (limited to 'Src/Modules/zftp.c')
| -rw-r--r-- | Src/Modules/zftp.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/Src/Modules/zftp.c b/Src/Modules/zftp.c index 80e09045c..de762e883 100644 --- a/Src/Modules/zftp.c +++ b/Src/Modules/zftp.c @@ -2678,14 +2678,15 @@ zfclose(int leaveparams) zfsendcmd("QUIT\r\n"); } 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. + * TCP fd. We need to do this before closing the FILE, since + * it's not usable afterwards. */ if (fileno(zfsess->cin) == zfsess->control->fd) zfsess->control->fd = -1; + fclose(zfsess->cin); zfsess->cin = NULL; } if (zfsess->control) { |
