[Pkg-javascript-commits] [uglifyjs] 10/228: in mangle_names there is a check that the variable name is legal and that it is not a reserved word. This should apply to propsmangle as well.

[Jonas Smedegaard]

This is an automated email from the git hooks/post-receive script.

js pushed a commit to branch master
in repository uglifyjs.

commit 81f1df14d7c931cddedad9666a7c57f4373ec918
Author: Anat Dagan <anadt at ironsrc.com>
Date:   Fri Feb 10 14:13:47 2017 +0200

    in mangle_names there is a check that the variable name is legal and that it is not a reserved word. This should apply to propsmangle as well.
---
 lib/propmangle.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/propmangle.js b/lib/propmangle.js
index f277747..3c75cac 100644
--- a/lib/propmangle.js
+++ b/lib/propmangle.js
@@ -149,6 +149,7 @@ function mangle_properties(ast, options) {
     // only function declarations after this line
 
     function can_mangle(name) {
+        if (!is_identifier(name)) return false;
         if (unmangleable.indexOf(name) >= 0) return false;
         if (reserved.indexOf(name) >= 0) return false;
         if (options.only_cache) {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/uglifyjs.git