Apply review suggestion

joefarebrother · michaelnebel · web-flow · commit b2d54842a6ce · 2023-06-23T17:00:52.000+01:00
Co-authored-by: Michael Nebel &lt;michaelnebel@github.com&gt;
diff --git a/csharp/ql/lib/semmle/code/csharp/security/auth/MissingFunctionLevelAccessControlQuery.qll b/csharp/ql/lib/semmle/code/csharp/security/auth/MissingFunctionLevelAccessControlQuery.qll
@@ -92,7 +92,7 @@ private predicate physicalRouteMatches(string route, string actual) {
   route = actual
   or
   route.charAt(0) = "~" and
-  exists(string dir | actual = dir + route.substring(1, route.length()) + ".cs")
+  exists(string dir | actual = dir + route.suffix(1) + ".cs")
 }
 
 /** An expression that indicates that some authorization/authentication check is being performed. */

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ private predicate physicalRouteMatches(string route, string actual) {`
`92`	`92`	`route = actual`
`93`	`93`	`or`
`94`	`94`	`route.charAt(0) = "~" and`
`95`		`- exists(string dir \| actual = dir + route.substring(1, route.length()) + ".cs")`
	`95`	`+ exists(string dir \| actual = dir + route.suffix(1) + ".cs")`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`/** An expression that indicates that some authorization/authentication check is being performed. */`