diff options
author | schmidt <douglascraigschmidt@users.noreply.github.com> | 2000-03-29 06:04:10 +0000 |
---|---|---|
committer | schmidt <douglascraigschmidt@users.noreply.github.com> | 2000-03-29 06:04:10 +0000 |
commit | 5be954be7973e55553ddd8084bfc6598ac19f909 (patch) | |
tree | a236f740660ade0fbb560df1015fbc8c7b25217c | |
parent | 5abbc343fc617931ddea47ee19866621e8aaf0f2 (diff) | |
download | ATCD-5be954be7973e55553ddd8084bfc6598ac19f909.tar.gz |
ChangeLogTag:Wed Mar 29 00:01:39 2000 Douglas C. Schmidt <schmidt@tango.cs.wustl.edu>
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | ChangeLogs/ChangeLog-02a | 10 | ||||
-rw-r--r-- | ChangeLogs/ChangeLog-03a | 10 | ||||
-rw-r--r-- | netsvcs/lib/Client_Logging_Handler.cpp | 5 |
4 files changed, 34 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog index 8fb83e84ff6..ffde6b8b5a7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,13 @@ +Wed Mar 29 00:01:39 2000 Douglas C. Schmidt <schmidt@tango.cs.wustl.edu> + + * netsvcs/lib/Client_Logging_Handler.cpp (handle_input): + ACE_OS::closesocket() was being used to do some very "low-level" + closing of handles without assigning a handle to + ACE_INVALID_HANDLE. This was causing problems. Therefore, we + now try to use this->peer ().close () if it's different from + handle. Thanks to Alexander Dergatch <dergy@websci.ru> for + reporting this. This fixes [BUGID 520]. + Tue Mar 28 12:13:12 2000 Nanbor Wang <nanbor@cs.wustl.edu> * ace/Containers_T.h: Fixed the mismatching filename in preamble. diff --git a/ChangeLogs/ChangeLog-02a b/ChangeLogs/ChangeLog-02a index 8fb83e84ff6..ffde6b8b5a7 100644 --- a/ChangeLogs/ChangeLog-02a +++ b/ChangeLogs/ChangeLog-02a @@ -1,3 +1,13 @@ +Wed Mar 29 00:01:39 2000 Douglas C. Schmidt <schmidt@tango.cs.wustl.edu> + + * netsvcs/lib/Client_Logging_Handler.cpp (handle_input): + ACE_OS::closesocket() was being used to do some very "low-level" + closing of handles without assigning a handle to + ACE_INVALID_HANDLE. This was causing problems. Therefore, we + now try to use this->peer ().close () if it's different from + handle. Thanks to Alexander Dergatch <dergy@websci.ru> for + reporting this. This fixes [BUGID 520]. + Tue Mar 28 12:13:12 2000 Nanbor Wang <nanbor@cs.wustl.edu> * ace/Containers_T.h: Fixed the mismatching filename in preamble. diff --git a/ChangeLogs/ChangeLog-03a b/ChangeLogs/ChangeLog-03a index 8fb83e84ff6..ffde6b8b5a7 100644 --- a/ChangeLogs/ChangeLog-03a +++ b/ChangeLogs/ChangeLog-03a @@ -1,3 +1,13 @@ +Wed Mar 29 00:01:39 2000 Douglas C. Schmidt <schmidt@tango.cs.wustl.edu> + + * netsvcs/lib/Client_Logging_Handler.cpp (handle_input): + ACE_OS::closesocket() was being used to do some very "low-level" + closing of handles without assigning a handle to + ACE_INVALID_HANDLE. This was causing problems. Therefore, we + now try to use this->peer ().close () if it's different from + handle. Thanks to Alexander Dergatch <dergy@websci.ru> for + reporting this. This fixes [BUGID 520]. + Tue Mar 28 12:13:12 2000 Nanbor Wang <nanbor@cs.wustl.edu> * ace/Containers_T.h: Fixed the mismatching filename in preamble. diff --git a/netsvcs/lib/Client_Logging_Handler.cpp b/netsvcs/lib/Client_Logging_Handler.cpp index 5bdae379842..a42cc7d74a8 100644 --- a/netsvcs/lib/Client_Logging_Handler.cpp +++ b/netsvcs/lib/Client_Logging_Handler.cpp @@ -157,7 +157,10 @@ ACE_Client_Logging_Handler::handle_input (ACE_HANDLE handle) "%n: %p\n", "remove_handler"), 0); - ACE_OS::closesocket (handle); + if (handle == this->peer ().get_handle ()) + this->peer ().close (); + else + ACE_OS::closesocket (handle); ACE_DEBUG ((LM_DEBUG, "client closing down\n")); return 0; |