Skip to content

Conversation

frankfliu
Copy link
Contributor

Fixes #3127

Description

Brief description of what this PR is about

  • If this change is a backward incompatible change, why must this change be made?
  • Interesting edge cases to note here

@xyang16
Copy link
Contributor

xyang16 commented Apr 29, 2024

This PR has a lot of other commits in it.

}
for (Path path : list) {
try {
Files.deleteIfExists(path);

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression

This path depends on a [user-provided value](1).
@frankfliu frankfliu merged commit c54db65 into deepjavalibrary:master Apr 30, 2024
@frankfliu frankfliu deleted the stream branch April 30, 2024 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resource Leak Issue with Files.list in ai.djl.util.Utils
2 participants