Bug 3129 - Notify MC unit tests crash in TSS cleanup
Summary: Notify MC unit tests crash in TSS cleanup
Status: RESOLVED FIXED
Alias: None
Product: TAO
Classification: Unclassified
Component: Notification Service (show other bugs)
Version: 1.6.1
Hardware: All All
: P3 normal
Assignee: ciju john
URL:
Depends on: 2980 3108
Blocks:
  Show dependency tree
 
Reported: 2007-11-07 09:49 CST by Johnny Willemsen
Modified: 2008-04-25 10:58 CDT (History)
1 user (show)

See Also:

Attachments
MC bug 2980 workaround (4.77 KB, text/x-patch)
2007-11-15 14:51 CST, ciju john 		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 Johnny Willemsen 2007-11-07 09:49:27 CST 
The Notification MC unit tests below crash in cleanup. This seems related to the reversal of 3108

+TAO/orbsvcs/tests/unit/Notify/MC/MonitorControlExt
+TAO/orbsvcs/tests/unit/Notify/MC/MonitorManager
+
Comment 1 Johnny Willemsen 2007-11-07 09:50:16 CST 
added 2908

Steve, can someone of OCI have a look, this seems related to the Service Config/Gestalt cleanup issue
Comment 2 Johnny Willemsen 2007-11-07 09:52:17 CST 
Crash with BCB

000146A9 ACE_Task_Flags
00480A1C ACE_TSS_Cleanup::thread_exit(this=:00AC7F2C)
004811E7 ACE_OS::cleanup_tss(main_thread=0)
004BA0B7 ACE_Thread_Adapter::invoke_i(this=:00AD8A50)
004B9F7C ACE_Thread_Adapter::invoke(this=:00AD8A50)
00428535 ace_thread_adapter(args=:00AD8A50)
Comment 3 Johnny Willemsen 2007-11-07 09:53:04 CST 
corrected depends
Comment 4 ciju john 2007-11-15 14:05:02 CST 
This crash is a manifestation of bug 2980 and will be gone once thats fixed. I have a workaround that fixes this issue for the Notification MonitorManager.

Ciju
Comment 5 ciju john 2007-11-15 14:51:35 CST 
Created attachment 870 [details]
MC bug 2980 workaround

Work around for bug 2980. Create the MC ORB in the parent thread
and pass to the ORBTask.
Setup a barrier to synch the parent and child threads. This
holds the parent thread till child initialization is completed.
Comment 6 ciju john 2008-04-25 10:58:22 CDT 
Workaround prevents crash.

Ciju