fix corner case in unsafe (#3380)
This commit is contained in:
@@ -3018,6 +3018,7 @@ merge(Compressor.prototype, {
|
|||||||
if (arg === value) return this;
|
if (arg === value) return this;
|
||||||
args.push(value);
|
args.push(value);
|
||||||
}
|
}
|
||||||
|
if (key == "replace" && typeof args[1] == "function") return this;
|
||||||
try {
|
try {
|
||||||
return val[key].apply(val, args);
|
return val[key].apply(val, args);
|
||||||
} catch (ex) {
|
} catch (ex) {
|
||||||
@@ -3139,6 +3140,7 @@ merge(Compressor.prototype, {
|
|||||||
} else if (expr instanceof AST_RegExp) {
|
} else if (expr instanceof AST_RegExp) {
|
||||||
map = native_fns.RegExp;
|
map = native_fns.RegExp;
|
||||||
} else if (expr.is_string(compressor)) {
|
} else if (expr.is_string(compressor)) {
|
||||||
|
if (this.property == "replace") return false;
|
||||||
map = native_fns.String;
|
map = native_fns.String;
|
||||||
} else if (!this.may_throw_on_access(compressor)) {
|
} else if (!this.may_throw_on_access(compressor)) {
|
||||||
map = native_fns.Object;
|
map = native_fns.Object;
|
||||||
|
|||||||
@@ -942,3 +942,21 @@ issue_2929: {
|
|||||||
}
|
}
|
||||||
expect_stdout: "PASS"
|
expect_stdout: "PASS"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
unsafe_string_replace: {
|
||||||
|
options = {
|
||||||
|
side_effects: true,
|
||||||
|
unsafe: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
"foo".replace("f", function() {
|
||||||
|
console.log("PASS");
|
||||||
|
});
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
"foo".replace("f", function() {
|
||||||
|
console.log("PASS");
|
||||||
|
});
|
||||||
|
}
|
||||||
|
expect_stdout: "PASS"
|
||||||
|
}
|
||||||
|
|||||||
@@ -1712,3 +1712,21 @@ unsafe_escaped: {
|
|||||||
}
|
}
|
||||||
expect_stdout: "PASS"
|
expect_stdout: "PASS"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
unsafe_string_replace: {
|
||||||
|
options = {
|
||||||
|
evaluate: true,
|
||||||
|
unsafe: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
"foo".replace("f", function() {
|
||||||
|
console.log("PASS");
|
||||||
|
});
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
"foo".replace("f", function() {
|
||||||
|
console.log("PASS");
|
||||||
|
});
|
||||||
|
}
|
||||||
|
expect_stdout: "PASS"
|
||||||
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user