You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2006/12/20 16:08:54 UTC

svn commit: r489109 - /james/server/trunk/src/java/org/apache/james/vut/AbstractVirtualUserTable.java

Author: norman
Date: Wed Dec 20 07:08:53 2006
New Revision: 489109

URL: http://svn.apache.org/viewvc?view=rev&rev=489109
Log:
Add missing synchronized to methods.

Modified:
    james/server/trunk/src/java/org/apache/james/vut/AbstractVirtualUserTable.java

Modified: james/server/trunk/src/java/org/apache/james/vut/AbstractVirtualUserTable.java
URL: http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/vut/AbstractVirtualUserTable.java?view=diff&rev=489109&r1=489108&r2=489109
==============================================================================
--- james/server/trunk/src/java/org/apache/james/vut/AbstractVirtualUserTable.java (original)
+++ james/server/trunk/src/java/org/apache/james/vut/AbstractVirtualUserTable.java Wed Dec 20 07:08:53 2006
@@ -278,7 +278,7 @@
     /**
      * @see org.apache.james.services.VirtualUserTableManagement#addMapping(java.lang.String, java.lang.String, java.lang.String)
      */
-    public boolean addMapping(String user, String domain, String mapping) throws InvalidMappingException {
+    public synchronized boolean addMapping(String user, String domain, String mapping) throws InvalidMappingException {
         String map = mapping.toLowerCase();
         
         if (map.startsWith(VirtualUserTable.ERROR_PREFIX)) {
@@ -296,7 +296,7 @@
     /**
      * @see org.apache.james.services.VirtualUserTableManagement#removeMapping(java.lang.String, java.lang.String, java.lang.String)
      */
-    public boolean removeMapping(String user, String domain, String mapping) throws InvalidMappingException {
+    public synchronized boolean removeMapping(String user, String domain, String mapping) throws InvalidMappingException {
         String map = mapping.toLowerCase();
     
         if (map.startsWith(VirtualUserTable.ERROR_PREFIX)) {
@@ -404,7 +404,7 @@
     /**
      * @see org.apache.james.services.VirtualUserTableManagement#addAliasDomainMapping(java.lang.String, java.lang.String)
      */
-    public boolean addAliasDomainMapping(String aliasDomain, String realDomain) throws InvalidMappingException {
+    public synchronized boolean addAliasDomainMapping(String aliasDomain, String realDomain) throws InvalidMappingException {
         getLogger().info("Add domain mapping: " + aliasDomain  + " => " + realDomain);
         return addMappingInternal(null, aliasDomain, VirtualUserTable.ALIASDOMAIN_PREFIX + realDomain);
     }
@@ -412,7 +412,7 @@
     /**
      * @see org.apache.james.services.VirtualUserTableManagement#removeAliasDomainMapping(java.lang.String, java.lang.String)
      */
-    public boolean removeAliasDomainMapping(String aliasDomain, String realDomain) throws InvalidMappingException {
+    public synchronized boolean removeAliasDomainMapping(String aliasDomain, String realDomain) throws InvalidMappingException {
         getLogger().info("Remove domain mapping: " + aliasDomain  + " => " + realDomain);
         return removeMappingInternal(null, aliasDomain, VirtualUserTable.ALIASDOMAIN_PREFIX + realDomain);
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org