You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@felix.apache.org by gn...@apache.org on 2012/03/15 17:43:08 UTC

svn commit: r1301076 - /felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java

Author: gnodet
Date: Thu Mar 15 16:43:08 2012
New Revision: 1301076

URL: http://svn.apache.org/viewvc?rev=1301076&view=rev
Log:
[FELIX-3396] Fileinstall is not able to handle configuration file if in a path containing braces

Modified:
    felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java

Modified: felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java
URL: http://svn.apache.org/viewvc/felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java?rev=1301076&r1=1301075&r2=1301076&view=diff
==============================================================================
--- felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java (original)
+++ felix/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java Thu Mar 15 16:43:08 2012
@@ -315,6 +315,9 @@ public class ConfigInstaller implements 
 
     Configuration findExistingConfiguration(String fileName) throws Exception
     {
+        // escape the special character in the filename
+        fileName = fileName.replace("(", "\\(");
+        fileName = fileName.replace(")", "\\)");
         String filter = "(" + DirectoryWatcher.FILENAME + "=" + fileName + ")";
         Configuration[] configurations = getConfigurationAdmin().listConfigurations(filter);
         if (configurations != null && configurations.length > 0)