Skip to content

Commit e826b8f

Browse files
committed
JS: update js/double-escaping message with escaping of \
1 parent 25b337b commit e826b8f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

javascript/ql/src/Security/CWE-116/DoubleEscaping.ql

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,9 +125,11 @@ from Replacement primary, Replacement supplementary, string message, string meta
125125
where
126126
primary.escapes(metachar, _) and
127127
supplementary = primary.getAnEarlierEscaping(metachar) and
128-
message = "may double-escape '" + metachar + "' characters from $@"
128+
message = "may double-escape '" + metachar.replaceAll("\\", "\\\\") + "' characters from $@"
129129
or
130130
primary.unescapes(_, metachar) and
131131
supplementary = primary.getALaterUnescaping(metachar) and
132-
message = "may produce '" + metachar + "' characters that are double-unescaped $@"
132+
message =
133+
"may produce '" + metachar.replaceAll("\\", "\\\\") +
134+
"' characters that are double-unescaped $@"
133135
select primary, "This replacement " + message + ".", supplementary, "here"

0 commit comments

Comments
 (0)