You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by kw...@apache.org on 2015/11/24 17:41:14 UTC
svn commit: r1716204 -
/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/logging/AppenderUtils.java
Author: kwall
Date: Tue Nov 24 16:41:14 2015
New Revision: 1716204
URL: http://svn.apache.org/viewvc?rev=1716204&view=rev
Log:
QPID-6884: [Java Broker] Fix path validation problem when Logger is given a filename without a directory portion
Modified:
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/logging/AppenderUtils.java
Modified: qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/logging/AppenderUtils.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/logging/AppenderUtils.java?rev=1716204&r1=1716203&r2=1716204&view=diff
==============================================================================
--- qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/logging/AppenderUtils.java (original)
+++ qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/logging/AppenderUtils.java Tue Nov 24 16:41:14 2015
@@ -97,7 +97,7 @@ public class AppenderUtils
static void validateLogFilePermissions(final File file)
{
- if ((file.exists() && (!file.isFile() || !file.canWrite())) || !file.getParentFile().canWrite())
+ if ((file.exists() && (!file.isFile() || !file.canWrite())) || !file.getAbsoluteFile().getParentFile().canWrite())
{
throw new IllegalConfigurationException(String.format("Do not have the permissions to log to file '%s'.", file.getAbsolutePath()));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org