Fix test comments

atorralba · atorralba · commit 1014400a0825 · 2021-06-17T15:03:45.000+02:00
diff --git a/java/ql/test/query-tests/security/CWE-749/UnsafeAndroidAccess.java b/java/ql/test/query-tests/security/CWE-749/UnsafeAndroidAccess.java
@@ -146,7 +146,6 @@ public boolean shouldOverrideUrlLoading(WebView view, String url) {
 		});
 
 		String thisUrl = getIntent().getStringExtra("url");
-		// This should be considered safe - the query lacks a proper sanitizer for partial URLs.
-		wv.loadUrl("https://www.mycorp.com/" + thisUrl);
+		wv.loadUrl("https://www.mycorp.com/" + thisUrl); // Safe
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,6 @@ public boolean shouldOverrideUrlLoading(WebView view, String url) {`
`146`	`146`	`});`
`147`	`147`
`148`	`148`	`String thisUrl = getIntent().getStringExtra("url");`
`149`		`- // This should be considered safe - the query lacks a proper sanitizer for partial URLs.`
`150`		`- wv.loadUrl("https://www.mycorp.com/" + thisUrl);`
	`149`	`+ wv.loadUrl("https://www.mycorp.com/" + thisUrl); // Safe`
`151`	`150`	`}`
`152`	`151`	`}`