Merge pull request #16 from ajaxorg/master

Proper escaping in text modules
mozilla · Nov 2, 2011 · 11ffb0e · 11ffb0e
2 parents 32f24d2 + 375d9a3
commit 11ffb0e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/dryice/index.js b/lib/dryice/index.js
@@ -677,8 +677,8 @@ copy.filter.addDefines = function(input, source) {
         source = source.path;
     }
 
-    input = input.replace(/"/g, '\\"');
-    input = '"' + input.replace(/\n/g, '" +\n  "') + '"';
+    input = input.replace(/\\/g, "\\\\").replace(/"/g, '\\"');
+    input = '"' + input.replace(/\n/g, '\\n" +\n  "') + '"';
 
     return 'define("text!' + source.toString() + '", [], ' + input + ');\n\n';
 };