fix up packageConfigPath regex

Author: guybedford

lib/utils.js

@@ -96,7 +96,7 @@ function getAlias(loader, canonicalName) {
       bestAliasSubpath = '';
       bestAliasLength = alias.split('/').length;
     }
-    else if (canonicalName.substr(0, mapped.length) == mapped && 
+    else if (canonicalName.substr(0, mapped.length) == mapped &&
         (canonicalName.length == mapped.length || canonicalName[mapped.length] == '/')) {
       bestAlias = alias;
       bestAliasSubpath = canonicalName.substr(mapped.length);
@@ -189,7 +189,7 @@ function getCanonicalNamePlain(loader, normalized, isPlugin) {
 
     // support trailing / in paths rules
     else if (curPath[curPath.length - 1] == '/' &&
-        normalized.substr(0, curPath.length - 1) == curPath.substr(0, curPath.length - 1) && 
+        normalized.substr(0, curPath.length - 1) == curPath.substr(0, curPath.length - 1) &&
         (normalized.length < curPath.length || normalized[curPath.length - 1] == curPath[curPath.length - 1])) {
       // first case is that canonicalize('src') = 'app' for 'app/': 'src/'
       return normalized.length < curPath.length ? p.substr(0, p.length - 1) : p + normalized.substr(curPath.length);
@@ -244,8 +244,7 @@ function createPkgConfigPathObj(path) {
   var length = Math.max(lastWildcard + 1, path.lastIndexOf('/'));
   return {
     length: length,
-    // NB handle regex control character escapes or simply create a test function here
-    regEx: new RegExp('^(' + path.substr(0, length).replace(/\*/g, '[^\\/]+') + ')(\\/|$)'),
+    regEx: new RegExp('^(' + path.substr(0, length).replace(/[.+?^${}()|[\]\\]/g, '\\$&').replace(/\*/g, '[^\\/]+') + ')(\\/|$)'),
     wildcard: lastWildcard != -1
   };
 }
@@ -373,4 +372,3 @@ exports.serializeCondition = serializeCondition;
 function serializeCondition(conditionObj) {
   return conditionObj.module + '|' + (conditionObj.negate ? '~' : '') + conditionObj.prop;
 }
-