GHSA-239w-m3h6-ch8v

## Summary

File Browser enforces per-user scope with `afero.NewBasePathFs(afero.NewOsFs(), scope)`, set up in `users/users.go`. This blocks lexical `../` traversal, but it does not stop the HTTP file handlers from following symbolic links before they open, serve, write, share, or list a file. As a result, a scoped user — and in some cases an unauthenticated public-share recipient — can cross the intended scope boundary by following a symlink whose path is lexically inside their scope but whose target is outside it.

Two distinct shapes are covered here:

- **Variant 1 — symlink as the final path component.** A symlink that lives inside the user's scoped tree and points to a file under the server root but outside the scope. The handlers record the symlink (`IsSymlink`) but then resolve and operate on the target anyway. - **Variant 2 — file or directory reached through a symlinked ancestor.** A regular file requested *through* a symlinked directory.

Read, write (including TUS resumable uploads), share creation, and public-share serving are all affected.

## Impact

In a multi-user deployment, if a symlink (a file symlink for Variant 1, or a directory symlink for Variant 2) exists inside a restricted user's scoped tree and resolves to a location outside that scope but reachable by the server process, the boundary can be crossed. Concretely, a user holding only normal File Browser permissions can:

- Read out-of-scope file contents and metadata via `GET /api/raw/{path}` and `GET /api/resources/{path}`. - Overwrite an out-of-scope target via `POST /api/resources/{path}?override=true`. - Overwrite or create an out-of-scope target via the TUS resumable upload path: `POST /api/tus/{path}?override=true` followed by `PATCH /api/tus/{path}`. - Create a public share for an out-of-scope target via `POST /api/share/{path}`, exposing it through `GET /api/public/dl/{hash}`.

For Variant 2, the same exposure reaches public-share recipients: a normal public directory share whose subtree contains a linked descendant lets an unauthenticated recipient read regular files behind the link, pull them into the share's archive download, and see the resolved target in directory listings.

This breaks the confidentiality and integrity guarantees that per-user scopes and password/anonymous shares are relied upon to provide, for any data the server process can reach.

## Technical details

Users are rooted with `afero.NewBasePathFs(afero.NewOsFs(), scope)`. Base-path rooting blocks lexical `../` traversal but does not prevent ordinary filesystem operations from following a symlink whose path is lexically inside the base.

The metadata layer records symlinks but does not consistently re-check the resolved target against the user's real scope:

- In `files/file.go`, `stat()` calls `LstatIfPossible`, sets `IsSymlink`, and only invokes the `WithinScope` containment check when `file.IsSymlink == true`. For Variant 1, this guard (where present) covers the final-element symlink; on the commit tested for Variant 1 the handler still resolved the target with `opts.Fs.Stat(opts.Path)` and served it. For Variant 2, `LstatIfPossible` follows a symlinked *ancestor* and returns the leaf as a regular file (`IsSymlink == false`), so `stat()` returns early and the scope check never runs at all. - `readListing` in `files/file.go` follows symlink entries to display the target's metadata. - `http/raw.go` builds a file object for the requested path and serves non-directories; its archive walker `getFiles` follows symlinks via `Stat`/`Open`, pulling linked descendants into archive downloads. - `http/resource.go` writes request bodies with `writeFile(d.user.Fs, r.URL.Path, ...)`, and the destination open follows symlinks. - `http/tus_handlers.go` (`tusPostHandler`, `tusPatchHandler`) calls `MkdirAll`/`OpenFile` on the request path directly with no containment check. Because a brand-new leaf does not stat an existing file, it skips the scope check entirely. - `http/share.go` stores a share for `r.URL.Path` without checking that the path is not a symlink escape; `http/public.go` later serves it for unauthenticated downloads (routed at `http/http.go:90-91`). - `http/data.go` applies dotfile and rule checks to the request path *string*, but never compares the resolved symlink target against the user's real scope.

## Proof of concept

### Variant 1 — symlink as final path component

Harness layout: server root is a temp directory; restricted user `restricted` is scoped to `/u1` with create, modify, rename, share, and download permissions; a second scope `/u2` holds the outside target `/u2/secret.txt` containing `other-secret`; and `/u1/link-out` is a symlink to `/u2/secret.txt`.

Confirmed bypasses (route-level tests against the real HTTP handlers):

- `GET /api/raw/link-out` → `200 OK`, body contains `other-secret` from `/u2/secret.txt`. - `POST /api/resources/link-out?override=true` → `200 OK`, `/u2/secret.txt` changed to `pwn`. - `POST /api/tus/link-out?override=true` → `201`, then `PATCH /api/tus/link-out` → `204`, `/u2/secret.txt` changed. - `POST /api/share/link-out` → `200 OK`, created a public share whose `GET /api/public/dl/{hash}` returned a body containing `other-secret`.

Minimal core of the read proof:

```go root := t.TempDir() os.MkdirAll(filepath.Join(root, "u1"), 0755) os.MkdirAll(filepath.Join(root, "u2"), 0755) os.WriteFile(filepath.Join(root, "u2", "secret.txt"), []byte("other-secret"), 0644) os.Symlink(filepath.Join(root, "u2", "secret.txt"), filepath.Join(root, "u1", "link-out"))

// restricted is a File Browser user scoped to /u1 with Download permission. rr := authenticatedRequest(t, restricted, http.MethodGet, "/api/raw/link-out", nil) if rr.Code != http.StatusOK || !strings.Contains(rr.Body.String(), "other-secret") { t.Fatalf("raw symlink exposed outside target: status=%d body=%q", rr.Code, rr.Body.String()) } ```

### Variant 2 — file reached through a symlinked ancestor

Authenticated scoped user whose scope contains a directory symlink `escape_link -> /srv/users/otheruser`:

``` # The symlink itself is correctly blocked GET /api/resources/escape_link -> 403 Forbidden

# A regular file THROUGH the symlinked directory is not GET /api/resources/escape_link/private.txt -> 200 OK {"content":"OTHER_USER_SECRET_DATA=...",...} GET /api/raw/escape_link/private.txt -> 200 OK OTHER_USER_SECRET_DATA=...

# Create/overwrite THROUGH the symlinked directory (TUS) POST /api/tus/escape_link/injected.txt (Upload-Length: 20) -> 201 Created PATCH /api/tus/escape_link/injected.txt (Upload-Offset: 0) -> 204 No Content (written into /srv/users/otheruser/) ```

Public directory share for `/shared`, where `/shared/link -> ../private` and `private/secret.txt` lives outside the share:

``` GET /api/public/dl/<hash>/link/secret.txt -> 200 OK symlink-secret GET /api/public/share/<hash>/link/secret.txt -> 200 OK {"path":"/link/secret.txt", ...} ```

Requesting the whole share as an archive pulls `link/secret.txt` into the zip, and listing the share root exposes the `link` entry with its resolved target metadata.

### Controls that held

The same harness confirmed that ordinary traversal is still rejected, so this is not generic `../` traversal:

- `GET /api/resources/../u2/secret.txt?checksum=sha256` did not succeed as the restricted user. - `GET /api/resources/%2e%2e/u2/secret.txt` did not succeed (encoded dot-dot). - `POST /api/resources/../u2/new.txt` did not create `/u2/new.txt`. - `PATCH /api/resources/own.txt?action=rename&destination=/../u2/moved.txt` did not move a file outside scope.

## Affected code

`users/users.go` (scope setup); `files/file.go` (`stat`, `readListing`); `http/raw.go` (`getFiles`); `http/resource.go` (`writeFile` destination); `http/tus_handlers.go` (`tusPostHandler`, `tusPatchHandler`); `http/share.go`; `http/public.go`; `http/http.go:90-91` (public routes); `http/data.go` (string-only path checks).

## Remediation

Resolve symlinks and verify that the resolved target remains inside the user's real scoped root before any file operation — serving, sharing, writing, truncating, renaming, copying, or deleting. Specifically:

- Call `WithinScope` (which resolves every path component with `filepath.EvalSymlinks`) for **all** paths in `stat()`, not only when the final element is a symlink. This closes the ancestor-symlink gap (Variant 2). - Add a `WithinScope` check before `MkdirAll`/`OpenFile` in `tusPostHandler` and `tusPatchHandler`, so a not-yet-existing leaf cannot skip containment. - Omit entries whose resolved target escapes the scope from `readListing`, and skip them in `getFiles` before stat/open/recursion. - Apply the same resolved-path check consistently to public share creation and public share serving. - As an alternative or defense-in-depth, reject symlinks for file operations unless an explicit administrator option enables them.

Add regression tests covering symlink reads, overwrites, TUS create/write, public shares (download, share-info, listing, and archive read paths), and the existing dot-dot controls — plus a positive test confirming that legitimately in-scope symlinks still resolve.

## Limitations and non-claims

- This is not generic `../` path traversal; dot-dot and encoded dot-dot controls held in the route-level tests. - This is not a proxy-auth confusion issue; the proofs use normal authenticated requests for a restricted user (and, for Variant 2's share case, an ordinary public-share recipient). - The proofs assume the relevant symlink already exists inside the scoped tree, or that another allowed workflow in the deployment can place it there — for example an SMB/NFS export, a Docker bind-mount, or an admin-created link. Web-UI-only creation of the symlink from scratch was not demonstrated.