From 8e34cd41f0a361d0f389aedb5527c0972986ab56 Mon Sep 17 00:00:00 2001 From: David Dykstra Date: Tue, 28 May 2002 15:42:51 +0000 Subject: [PATCH] Close previously opened file descriptor when mkstemp fails in recv_files(). Every other failure condition in that function was alreadying doing this, and I saw a case with a lot of "mkstemp...No space left on device" messages started becoming "mkstemp...Too many open files" messages because of this bug. Not that it makes a whole lot of difference, since nothing gets copied because the disk was out of space. --- receiver.c | 1 + 1 file changed, 1 insertion(+) diff --git a/receiver.c b/receiver.c index 14282338..a4a3383d 100644 --- a/receiver.c +++ b/receiver.c @@ -429,6 +429,7 @@ int recv_files(int f_in,struct file_list *flist,char *local_name,int f_gen) rprintf(FERROR,"mkstemp %s failed: %s\n",fnametmp,strerror(errno)); receive_data(f_in,buf,-1,NULL,file->length); if (buf) unmap_file(buf); + if (fd1 != -1) close(fd1); continue; } -- 2.34.1