Headline
CVE-2022-36007: Merge pull request #4 from BulkSecurityGeneratorProjectV2/fix/JLL/par… · jlangch/venice@215ae91
Venice is a Clojure inspired sandboxed Lisp dialect with excellent Java interoperability. A partial path traversal issue exists within the functions load-file
and load-resource
. These functions can be limited to load files from a list of load paths. Assuming Venice has been configured with the load paths: [ "/Users/foo/resources" ]
When passing relative paths to these two vulnerable functions everything is fine: (load-resource "test.png")
=> loads the file “/Users/foo/resources/test.png” (load-resource "../resources-alt/test.png")
=> rejected, outside the load path When passing absolute paths to these two vulnerable functions Venice may return files outside the configured load paths: (load-resource "/Users/foo/resources/test.png")
=> loads the file “/Users/foo/resources/test.png” (load-resource "/Users/foo/resources-alt/test.png")
=> loads the file “/Users/foo/resources-alt/test.png” !!! The latter call suffers from the Partial Path Traversal vulnerability. This issue’s scope is limited to absolute paths whose name prefix matches a load path. E.g. for a load-path "/Users/foo/resources"
, the actor can cause loading a resource also from "/Users/foo/resources-alt"
, but not from "/Users/foo/images"
. Versions of Venice before and including v1.10.17 are affected by this issue. Upgrade to Venice >= 1.10.18, if you are on a version < 1.10.18. There are currently no known workarounds.
Permalink
Browse files
Merge pull request #4 from BulkSecurityGeneratorProjectV2/fix/JLL/par…
…tial-path-traversal-vulnerability
[SECURITY] Fix Partial Path Traversal Vulnerability
- Loading branch information
2 parents 4b2fb36 + c942c73 commit 215ae91bb964013b0a2d70718a692832d561ae0a
Showing 1 changed file with 1 addition and 1 deletion.
@@ -258,7 +258,7 @@ private boolean isFileWithinDirectory(
if (dir_.isDirectory()) {
final File fl = new File(dir_, file.getPath());
if (fl.isFile()) {
if (fl.getCanonicalPath().startsWith(dir_.getCanonicalPath())) {
if (fl.getCanonicalFile().toPath().startsWith(dir_.getCanonicalFile().toPath())) {
// Prevent accessing files outside the load-path.
// E.g.: …/…/coffee
return true;
0 comments on commit 215ae91
Please sign in to comment.
Related news
### Impact A partial path traversal issue exists within the functions `load-file` and `load-resource`. These functions can be limited to load files from a list of load paths. Assuming Venice has been configured with the load paths: `[ "/Users/foo/resources" ]` When passing **relative** paths to these two vulnerable functions everything is fine: `(load-resource "test.png")` => loads the file "/Users/foo/resources/test.png" `(load-resource "../resources-alt/test.png")` => rejected, outside the load path When passing **absolute** paths to these two vulnerable functions Venice may return files outside the configured load paths: `(load-resource "/Users/foo/resources/test.png")` => loads the file "/Users/foo/resources/test.png" `(load-resource "/Users/foo/resources-alt/test.png")` => loads the file "/Users/foo/resources-alt/test.png" !!! The latter call suffers from the _Partial Path Traversal_ vulnerability. This issue’s scope is limited to absolute paths whose na...