[exo-jcr-commits] exo-jcr SVN: r5672 - jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl.

do-not-reply at jboss.org do-not-reply at jboss.org
Mon Feb 20 10:01:21 EST 2012


Author: andrew.plotnikov
Date: 2012-02-20 10:01:20 -0500 (Mon, 20 Feb 2012)
New Revision: 5672

Modified:
   jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java
Log:
EXOJCR-1767: Fixed BackupManager throws NPE in case of wrong configuration of backup logs folder

Modified: jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java
===================================================================
--- jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java	2012-02-20 14:26:21 UTC (rev 5671)
+++ jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java	2012-02-20 15:01:20 UTC (rev 5672)
@@ -1201,7 +1201,11 @@
       logsDirectory = new File(backupDir);
       if (!PrivilegedFileHelper.exists(logsDirectory))
       {
-         PrivilegedFileHelper.mkdirs(logsDirectory);
+         if (!PrivilegedFileHelper.mkdirs(logsDirectory))
+         {
+            throw new IllegalStateException("Could not create the backup directory at "
+               + logsDirectory.getAbsolutePath());
+         }
       }
 
       if (defIncrPeriod == null)



More information about the exo-jcr-commits mailing list