You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by fe...@apache.org on 2011/03/20 22:49:21 UTC

svn commit: r1083588 - /directory/shared/trunk/ldap/codec/standalone/src/main/java/org/apache/directory/shared/ldap/codec/standalone/StandaloneLdapCodecService.java

Author: felixk
Date: Sun Mar 20 21:49:21 2011
New Revision: 1083588

URL: http://svn.apache.org/viewvc?rev=1083588&view=rev
Log:
Respect return value of mkdirs

Modified:
    directory/shared/trunk/ldap/codec/standalone/src/main/java/org/apache/directory/shared/ldap/codec/standalone/StandaloneLdapCodecService.java

Modified: directory/shared/trunk/ldap/codec/standalone/src/main/java/org/apache/directory/shared/ldap/codec/standalone/StandaloneLdapCodecService.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/codec/standalone/src/main/java/org/apache/directory/shared/ldap/codec/standalone/StandaloneLdapCodecService.java?rev=1083588&r1=1083587&r2=1083588&view=diff
==============================================================================
--- directory/shared/trunk/ldap/codec/standalone/src/main/java/org/apache/directory/shared/ldap/codec/standalone/StandaloneLdapCodecService.java (original)
+++ directory/shared/trunk/ldap/codec/standalone/src/main/java/org/apache/directory/shared/ldap/codec/standalone/StandaloneLdapCodecService.java Sun Mar 20 21:49:21 2011
@@ -247,7 +247,12 @@ public class StandaloneLdapCodecService 
         }
         else if ( ! this.pluginDirectory.exists() )
         {
-            this.pluginDirectory.mkdirs();
+            if ( ! this.pluginDirectory.mkdirs() )
+            {
+                String msg = "The provided plugin directory is not creatable:" + this.pluginDirectory.getAbsolutePath();
+                LOG.error( msg );
+                throw new IllegalArgumentException( msg );
+            }
         }
         
         if ( this.pluginDirectory == null )