991 2014-09-08 03:51:27 +0000 JacORBConfiguration's getAttributeXXX methods are not thread safe 2014-11-07 05:05:56 +0000 1 1 1 Unclassified JacORB ORB 3.5 PC Windows NT RESOLVED FIXED P5 critical --- 1 franky.bugzilla jacorb-bugs jacorb oldest_to_newest 3200 0 445 franky.bugzilla 2014-09-08 03:51:27 +0000 Created attachment 445 thread dump I have a setup where I connect to multiple CORBA enabled devices in parallel from a thread pool. Once in a while I see that a number of threads used for the CORBA communication are stuck in an endless loop (threads are running at 100%) at start up of my application. A thread dump of a stuck thread can be found in the attached file. The threads got stuck in HashMap.getEntry which is known behavior when accessed and modified concurrently and the owner of this HashMap is JacOrbConfiguration. In fact there are 3 HashMaps which can be accessed concurrently in JacOrbConfiguration being stringAttributes, booleanAttributes and numberAttributes. The problem is that some of the methods getAttribute, getAttributeAsBoolean, getAttributeAsInteger try to get the value from the HashMap and if not the default value is put into the HashMap. If those getAttributeXXX methods are called concurrently the HashMaps can be accessed and modified concurrently and result in an infinite loop. I encountered this on 3.3 but I guess it's also on 3.5 as I didn't find any bug report related to this. 3205 1 franky.bugzilla 2014-10-01 08:47:36 +0000 I've encountered this problem on 3.3 but I see the JacORBConfiguration code in 3.5 is the same in this area, meaning the problem is also present in 3.5 3208 2 jacorb 2014-10-02 03:59:45 +0000 Do you have a thread dump? 3216 3 jacorb 2014-11-07 05:05:56 +0000 Hopefully fixed by SHA 6570aa2109ac3506c14fd928358107a66a5b7d23 445 2014-09-08 03:51:27 +0000 2014-09-08 03:51:27 +0000 thread dump threaddump.txt text/plain 1138 franky.bugzilla amF2YS5sYW5nLlRocmVhZC5TdGF0ZTogUlVOTkFCTEUNCglhdCBqYXZhLnV0aWwuSGFzaE1hcC5n ZXRFbnRyeShVbmtub3duIFNvdXJjZSkNCglhdCBqYXZhLnV0aWwuSGFzaE1hcC5nZXQoVW5rbm93 biBTb3VyY2UpDQoJYXQgb3JnLmphY29yYi5jb25maWcuSmFjT1JCQ29uZmlndXJhdGlvbi5nZXRB dHRyaWJ1dGVBc0Jvb2xlYW4oSmFjT1JCQ29uZmlndXJhdGlvbi5qYXZhOjExMTkpDQoJYXQgb3Jn LmphY29yYi5jb25maWcuSmFjT1JCQ29uZmlndXJhdGlvbi5nZXRMb2dnZXIoSmFjT1JCQ29uZmln dXJhdGlvbi5qYXZhOjc2NykNCglhdCBvcmcuamFjb3JiLm9yYi5DRFJJbnB1dFN0cmVhbS5jb25m aWd1cmUoQ0RSSW5wdXRTdHJlYW0uamF2YTozMDgpDQoJYXQgb3JnLmphY29yYi5vcmIuQ0RSSW5w dXRTdHJlYW0uPGluaXQ+KENEUklucHV0U3RyZWFtLmphdmE6MjM0KQ0KCWF0IG9yZy5qYWNvcmIu b3JiLkNEUklucHV0U3RyZWFtLjxpbml0PihDRFJJbnB1dFN0cmVhbS5qYXZhOjI0NykNCglhdCBv cmcuamFjb3JiLm9yYi5DRFJJbnB1dFN0cmVhbS48aW5pdD4oQ0RSSW5wdXRTdHJlYW0uamF2YToy NzkpDQoJYXQgb3JnLmphY29yYi5vcmIuZXRmLlByb2ZpbGVCYXNlLmluaXRGcm9tUHJvZmlsZURh dGEoUHJvZmlsZUJhc2UuamF2YToyOTQpDQoJYXQgb3JnLmphY29yYi5vcmIuZXRmLlByb2ZpbGVC YXNlLmRlbWFyc2hhbChQcm9maWxlQmFzZS5qYXZhOjIxMSkNCglhdCBvcmcuamFjb3JiLm9yYi5p aW9wLklJT1BGYWN0b3JpZXMuZGVtYXJzaGFsX3Byb2ZpbGUoSUlPUEZhY3Rvcmllcy5qYXZhOjQ5 KQ0KCWF0IG9yZy5qYWNvcmIub3JiLmlpb3AuSUlPUEZhY3Rvcmllcy5kZW1hcnNoYWxfcHJvZmls ZShJSU9QRmFjdG9yaWVzLmphdmE6MzYpDQoJYXQgb3JnLmphY29yYi5vcmIuUGFyc2VkSU9SLmRl Y29kZShQYXJzZWRJT1IuamF2YTozMzMpDQoJYXQgb3JnLmphY29yYi5vcmIuUGFyc2VkSU9SLnBh cnNlX3N0cmluZ2lmaWVkX2lvcihQYXJzZWRJT1IuamF2YTo2ODMpDQoJYXQgb3JnLmphY29yYi5v cmIuUGFyc2VkSU9SLnBhcnNlKFBhcnNlZElPUi5qYXZhOjU4NikNCglhdCBvcmcuamFjb3JiLm9y Yi5QYXJzZWRJT1IuPGluaXQ+KFBhcnNlZElPUi5qYXZhOjI2MykNCgkuLi4gPG93biBjb2RlPg==