diff options
author | bala <balanatarajan@users.noreply.github.com> | 2002-11-18 17:33:40 +0000 |
---|---|---|
committer | bala <balanatarajan@users.noreply.github.com> | 2002-11-18 17:33:40 +0000 |
commit | 06b2ba6669db59372e244cba354c4d31ea9ead19 (patch) | |
tree | c7a83540476c362b5949c077b9830405887c370a | |
parent | db174e7f2c66eb164d209fce3bc031a9163d6974 (diff) | |
download | ATCD-06b2ba6669db59372e244cba354c4d31ea9ead19.tar.gz |
ChangeLogTag: Mon Nov 18 12:27:46 2002 Balachandran Natarajan <bala@isis-server.isis.vanderbilt.edu>
-rw-r--r-- | ChangeLog | 12 | ||||
-rw-r--r-- | ChangeLogs/ChangeLog-03a | 12 | ||||
-rw-r--r-- | ace/Dev_Poll_Reactor.cpp | 2 | ||||
-rw-r--r-- | ace/Select_Reactor_T.cpp | 2 | ||||
-rw-r--r-- | ace/WFMO_Reactor.i | 2 |
5 files changed, 27 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog index f9da9c37627..d9bb5b2801d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +Mon Nov 18 12:27:46 2002 Balachandran Natarajan <bala@isis-server.isis.vanderbilt.edu> + + * ace/Dev_Poll_Reactor.cpp (handler): + * ace/WFMO_Reactor.i (handler): + * ace/Select_Reactor_Base_T.cpp (handler_i): The methods that + returns a signal handler associated with a certain type of + signal, does restrictive sanity checks. One of the checks that + it performs is to see whether a pointer to an event_handler is 0 + just before assigning a new event handler pointer. This check + was not required at all and hence it is now removed. Thanks to + Edan Ayal for reporting this. + Sat Nov 16 09:38:38 2002 Balachandran Natarajan <bala@isis-server.isis.vanderbilt.edu> * ace/*/Makefile: diff --git a/ChangeLogs/ChangeLog-03a b/ChangeLogs/ChangeLog-03a index f9da9c37627..d9bb5b2801d 100644 --- a/ChangeLogs/ChangeLog-03a +++ b/ChangeLogs/ChangeLog-03a @@ -1,3 +1,15 @@ +Mon Nov 18 12:27:46 2002 Balachandran Natarajan <bala@isis-server.isis.vanderbilt.edu> + + * ace/Dev_Poll_Reactor.cpp (handler): + * ace/WFMO_Reactor.i (handler): + * ace/Select_Reactor_Base_T.cpp (handler_i): The methods that + returns a signal handler associated with a certain type of + signal, does restrictive sanity checks. One of the checks that + it performs is to see whether a pointer to an event_handler is 0 + just before assigning a new event handler pointer. This check + was not required at all and hence it is now removed. Thanks to + Edan Ayal for reporting this. + Sat Nov 16 09:38:38 2002 Balachandran Natarajan <bala@isis-server.isis.vanderbilt.edu> * ace/*/Makefile: diff --git a/ace/Dev_Poll_Reactor.cpp b/ace/Dev_Poll_Reactor.cpp index 620a38c1e8c..6fb01fe5daa 100644 --- a/ace/Dev_Poll_Reactor.cpp +++ b/ace/Dev_Poll_Reactor.cpp @@ -2119,7 +2119,7 @@ ACE_Dev_Poll_Reactor::handler (int signum, if (handler == 0) return -1; - else if (eh != 0 && *eh != 0) + else if (eh != 0) *eh = handler; return 0; diff --git a/ace/Select_Reactor_T.cpp b/ace/Select_Reactor_T.cpp index 9c42cd930b2..ed1332ece07 100644 --- a/ace/Select_Reactor_T.cpp +++ b/ace/Select_Reactor_T.cpp @@ -82,7 +82,7 @@ ACE_Select_Reactor_T<ACE_SELECT_REACTOR_TOKEN>::handler_i (int signum, if (handler == 0) return -1; - else if (eh != 0 && *eh != 0) + else if (eh != 0) *eh = handler; return 0; } diff --git a/ace/WFMO_Reactor.i b/ace/WFMO_Reactor.i index 8db81e4808b..98c6c5d81ba 100644 --- a/ace/WFMO_Reactor.i +++ b/ace/WFMO_Reactor.i @@ -1005,7 +1005,7 @@ ACE_WFMO_Reactor::handler (int signum, ACE_Event_Handler **eh) if (handler == 0) return -1; - else if (eh != 0 && *eh != 0) + else if (eh != 0) *eh = handler; return 0; } |