github / github/codeql

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

Open
#3,709 0 comments 0 reactions 0 assignees View on GitHub
Java question
Dominant language
CodeQL
Stars
10.1k
Forks
2.1k
Avg merge
2d 15h
Merged PRs (30d)
141

Description

**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/

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.