Bug 1592 - postinvoke is called after the send_reply/send_other/send_exception interception points
Summary: postinvoke is called after the send_reply/send_other/send_exception intercept...
Status: ASSIGNED
Alias: None
Product: TAO
Classification: Unclassified
Component: Portable Interceptors (show other bugs)
Version: 1.6.9
Hardware: x86 Windows 2000
: P3 normal
Assignee: DOC Center Support List (internal)
URL:
Depends on: 663 1369
Blocks: 2511
  Show dependency tree
 
Reported: 2003-09-11 04:21 CDT by Hans Bos
Modified: 2009-06-19 04:32 CDT (History)
0 users

See Also:

Attachments
PI / ServantLocator regression test. (11.16 KB, application/octet-stream)
2003-09-22 13:26 CDT, Ossama Othman 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Hans Bos 2003-09-11 04:21:53 CDT 
The postinvoke of a request is called after the send_reply interceptor for that 
request is called.

In the Corba specification 3.0.1 at 21.3.10.2, the order for calling filters 
and pre-/postinvoke is:

1. ServerRequestInterceptor.receive_request_service_contexts;
2. ServantLocator.preinvoke;
3. ServerRequestInterceptor.receive_request
4. the operation
5. ServantLocator.postinvoke;
6. ServerRequestInterceptor send_reply, send_exception, or send_other.

In TAO 5 and 6 are exchanged.
Comment 1 Ossama Othman 2003-09-22 13:23:33 CDT 
I've written a regression test that confirms this bug.  It will be fixed once
the code refactoring CVS branch is merged with the HEAD branch since the changes
on that branch will nullify any changes I make.
Comment 2 Ossama Othman 2003-09-22 13:26:57 CDT 
Created attachment 237 [details]
PI / ServantLocator regression test.
Comment 3 Ossama Othman 2003-09-22 16:18:08 CDT 
Proper interceptor flow cannot occur in all circumstances as long as the bug
described bug 663 remains unresolved.
Comment 4 Johnny Willemsen 2005-03-24 04:28:34 CST 
Ossama, do you have the test on your system available for a commit or do we 
need to get it from the attachment? With all server code refactoring it seems 
to me that it should be doable to fix this.
Comment 5 Johnny Willemsen 2007-02-21 08:24:14 CST 
added regression to the repo, this is not fixed yet.

Wed Feb 21 14:21:12 UTC 2007  Johnny Willemsen  <jwillemsen@remedy.nl>

        * tests/POA/Bug_1592_Regression/*:
          New test program for bug 1592, not fixed yet.

        *
tests/POA/Excessive_Object_Deactivations/Excessive_Object_Deactivations.mpc:
        * tests/POA/MT_Servant_Locator/MT_Servant_Locator.mpc:
          Simplified
Comment 6 Johnny Willemsen 2007-02-21 08:43:02 CST 
ServantLocator.postinvoke is called from ~Servant Upcall, the methods from step
6 are invoked around the invocation, this will need some structural changes in
the portableserver library
Comment 7 Johnny Willemsen 2007-05-21 06:56:02 CDT 
still valid
Comment 8 Johnny Willemsen 2007-08-27 08:31:23 CDT 
still failing with x.5.10
Comment 9 Johnny Willemsen 2008-02-18 13:50:01 CST 
still valid
Comment 10 Johnny Willemsen 2009-06-19 04:32:31 CDT 
still valid