Apply suggestions from code review

aschackmull · web-flow · commit 6ffaa6918a5c · 2022-09-06T14:11:48.000+02:00
diff --git a/java/ql/src/experimental/Security/CWE/CWE-625/PermissiveDotRegex.qhelp b/java/ql/src/experimental/Security/CWE/CWE-625/PermissiveDotRegex.qhelp
@@ -4,7 +4,7 @@
 <qhelp>
 
 <overview>
-<p>By default, a "dot" (<code>.</code>) in a regular expression matches all characters except the new line characters <code>\n</code> and
+<p>By default, a "dot" (<code>.</code>) in a regular expression matches all characters except the newline characters <code>\n</code> and
 <code>\r</code>. Regular expressions containing a dot can be bypassed with the characters <code>\r</code>(<code>%0a</code>) and 
 <code>\n</code>(<code>%0d</code>) when the default Java regular expression matching implementations are used. This becomes a security issue
 if these regular expressions are used to decide whether to grant access to protected application resources.</p>
diff --git a/java/ql/src/experimental/Security/CWE/CWE-625/PermissiveDotRegexQuery.qll b/java/ql/src/experimental/Security/CWE/CWE-625/PermissiveDotRegexQuery.qll
@@ -19,7 +19,7 @@ private class PermissiveDotStr extends StringLiteral {
   }
 }
 
-/** Remote flow sources obtained from the URI of a serlvet request. */
+/** Remote flow sources obtained from the URI of a servlet request. */
 private class GetServletUriSource extends SourceModelCsv {
   override predicate row(string row) {
     row =

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ private class PermissiveDotStr extends StringLiteral {`
`19`	`19`	`}`
`20`	`20`	`}`
`21`	`21`
`22`		`-/** Remote flow sources obtained from the URI of a serlvet request. */`
	`22`	`+/** Remote flow sources obtained from the URI of a servlet request. */`
`23`	`23`	`private class GetServletUriSource extends SourceModelCsv {`
`24`	`24`	`override predicate row(string row) {`
`25`	`25`	`row =`