drew
/
acadia-newlib
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Cygwin: don't copy path_conv in fhandler_base::reset 

There's a slim chance that duplicating fhandlers may end up duplicating
path_conv_handle handles twice ending up with a handle leak, due to
fhandler_base::reset calling path_conv::operator<< after the only
caller, fhandler::copyto, already called path_conv::operator=.

Just drop the call which basically duplicates what path_conv::operator=
already did.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
This commit is contained in:
Corinna Vinschen 2021-02-10 10:33:36 +01:00
parent 89b3833211
commit c875ed3744
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
winsup/cygwin/fhandler.cc
View File

@ -43,7 +43,6 @@ LONG NO_COPY fhandler_base_overlapped::asio_close_counter;
void void
fhandler_base::reset (const fhandler_base *from) fhandler_base::reset (const fhandler_base *from)
{ {
pc << from->pc;
ra.rabuf = NULL; ra.rabuf = NULL;
ra.ralen = 0; ra.ralen = 0;
ra.raixget = 0; ra.raixget = 0;