Skip to content

Commit 60d5ed9

Browse files
committed
Input from Review
1 parent 40c3b54 commit 60d5ed9

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

java/ql/src/experimental/Security/CWE/CWE-036/OpenStream.qhelp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,10 @@ is called on the URL, potentially leading to a local file access.</p>
2323
</example>
2424

2525
<references>
26-
<ul>
2726
<li>Java Platform, Standard Edition 11, API Specification:
2827
<a href="https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/net/URL.html">
2928
Class URL</a>.
3029
</li>
31-
</ul>
3230
<!-- LocalWords: CWE -->
3331
</references>
3432

java/ql/src/experimental/Security/CWE/CWE-036/OpenStream.ql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ class URLConstructor extends ClassInstanceExpr {
1717
Expr stringArg() {
1818
// Query only in URL's that were constructed by calling the single parameter string constructor.
1919
this.getConstructor().getNumberOfParameters() = 1 and
20-
this.getConstructor().getParameter(0).getType() instanceof TypeString
21-
and result = this.getArgument(0)
20+
this.getConstructor().getParameter(0).getType() instanceof TypeString and
21+
result = this.getArgument(0)
2222
}
2323
}
2424

0 commit comments

Comments
 (0)