[jboss-cvs] JBossAS SVN: r90334 - projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins.

[jboss-cvs-commits at lists.jboss.org]

Author: kabir.khan at jboss.com
Date: 2009-06-17 10:36:03 -0400 (Wed, 17 Jun 2009)
New Revision: 90334

Modified:
   projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
Log:
[JBKERNEL-25] R/W lock getting/setting Executor

Modified: projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java
===================================================================
--- projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2009-06-17 14:22:43 UTC (rev 90333)
+++ projects/kernel/trunk/dependency/src/main/java/org/jboss/dependency/plugins/AbstractController.java	2009-06-17 14:36:03 UTC (rev 90334)
@@ -129,7 +129,15 @@
    public void setExecutor(Executor executor)
    {
       // TODO - security
-      this.executor = executor;
+      lockWrite();
+      try
+      {
+         this.executor = executor;
+      }
+      finally
+      {
+         unlockWrite();
+      }
    }
 
    /**
@@ -150,7 +158,15 @@
     */
    protected Executor getExecutionEnvironment()
    {
-      return executor;
+      lockRead();
+      try
+      {
+         return executor;
+      }
+      finally
+      {
+         unlockRead();
+      }
    }
 
    public boolean isShutdown()