[ARTEMIS-274] Make ActiveMQThreadFactory instantiation running within doPrivileged block

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.0
• Fix Version/s: 1.2.0
• Assignee: Unassigned
• Reporter: Ivo Studensky
• Created: 22/Oct/15 12:19
• Updated: 08/Jun/16 13:27
• Resolved: 22/Oct/15 14:37

Description

At the moment, ActiveMQThreadFactory caches AccessControlContext inside of its constructor and uses it later in newThread() method invocation to create a thread inside of doPrivileged block. This leads to not enough permissions issue when a client is trying to create a new connection on a factory taken from JNDI, see https://issues.jboss.org/browse/WFLY-5166.

To fix it all the invocations of the factory constructor should run within a privileged block to get a privileged ACC to the factory. This change should be similar to how it works in WildFly right now.