github / github/codeql

Catch clauses should (maybe) not be enclosed by their try stmt

Offen
#3,709 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Java question
Vorherrschende Sprache
CodeQL
Sterne
10.1k
Forks
2.1k
Ø Merge
2 T. 15 Std.
Gemergte PRs (30 T.)
141

Beschreibung

**Description of the issue**
CodeQL models `CatchClause`s as being part of `TryStmt`s.
I personally find this counter-intuitive since the `{}` braces of the `try {}` block do not enclose the `catch() {}`in any way.
```java
try {
} catch() {
}
```
Here `catch(){}` would be enclosed by `try {}`
```codeql
import java

from CatchClause c, TryStmt try
where c.getEnclosingStmt() = try
select c, try
```
Link to query:
https://lgtm.com/query/2349457352250693633/

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.