Dokima Report
Severity
Critical: 0 · High: 6 · Medium: 17 · Low: 30 · Dispositioned: 4
Active Issues
| Severity | Issue | Repository | Component | Confidence |
|---|---|---|---|---|
| high Fully Valid | File-event hooks substitute attacker-controlled filenames into command strings before shell parsing, enabling low-privilege command injection that bypasses Perm.Execute When file-event hooks are turned on, the server pastes the name of the file a user just uploaded, renamed, or deleted directly into a command that runs on the server. A user who simply names their file with shell tricks can make the server run their own commands, even without permission to run commands. | filebrowser | command-execution | high |
| high Fully Valid | Lossy username normalization for home-directory derivation lets a distinct signup account share a victim's home directory Usernames are stored exactly as typed, but the folder for each user is named after a cleaned-up version of the username. Two different usernames can clean to the same folder name, so someone can register a look-alike name and end up sharing another person's private files. | filebrowser | settings-config | high |
| high Possibly Valid | afero BasePathFs confines by textual prefix only and follows symlinks, so an in-scope symlink escapes the user root on read and write Each user is meant to stay inside their own folder, but the confinement check only compares text, it does not resolve symbolic links. If a shortcut pointing outside the folder exists inside a user's area, the user can read or overwrite files anywhere the server can reach — including the app's database. | filebrowser | filesystem-path-resolution | medium |
| high Possibly Valid | Shipped default scope "." resolves to server root, granting self-registered accounts whole-tree read/write/delete With the built-in default settings, anyone who signs up for an account is placed at the very top of the server's file tree instead of a private folder of their own. That gives every self-registered user permission to read, change, and delete all served files. | filebrowser | settings-config | medium |
| high Possibly Valid | Recursive out-of-scope deletion via symlinked path components in the delete/move sinks When a user deletes a file or folder, the app trusts the path text but does not check whether any folder along the way is a shortcut that points outside the user's allowed area. If such a shortcut exists, the delete can wipe out unrelated files elsewhere on the server, including the app's own login database. | filebrowser | filesystem-path-resolution | medium |
| high Possibly Valid | Out-of-scope file read/exfiltration via symlinked path components in the rename/move sink When a user renames or moves a file, the app checks the path as plain text but doesn't verify that folders along the way are real folders rather than shortcuts pointing elsewhere. An attacker can use such a shortcut to pull a file from outside their allowed area into it and then download it, or push their files out. | filebrowser | filesystem-path-resolution | medium |
| medium Fully Valid | POST override error-rollback recursively deletes a directory, bypassing the Perm.Delete gate A user who is allowed to create and modify files but not to delete them can still wipe out an entire folder. Uploading over an existing directory triggers an error-cleanup step that recursively deletes that directory, doing exactly what the missing delete permission was supposed to prevent. | filebrowser | resource-file-crud | high |
| medium Fully Valid | Attacker-controlled EXIF thumbnail-size drives unbounded (~4GB) allocation during JPEG thumbnail preview A specially crafted image can make the server try to reserve several gigabytes of memory just from reading its metadata. Any logged-in user allowed to download files can request a thumbnail of such an image and exhaust the server's memory, potentially crashing it for everyone. | filebrowser | preview-thumbnails | high |
| medium Fully Valid | Slice-bounds-out-of-range panic in share cleanup loop (mutating slice while ranging) denies share-listing endpoints Listing shares can crash the request whenever two or more shares in the queried set have expired, because a cleanup loop deletes entries while looping over the list incorrectly. A low-privilege user can trigger this on demand and also break the admin view that lists everyone's shares. | filebrowser | storage-persistence | high |
| medium Fully Valid | Unbounded req.Which field list in userPutHandler amplifies one request into N fsync'd DB transactions (+N bcrypt) A single request to update a user account can name the same field thousands of times, and the server performs one separate disk-syncing database write for each entry (and a full password hash for each 'Password' entry). One authenticated user can turn one request into massive disk and CPU work. | filebrowser | users-management | high |
| medium Possibly Valid | Image preview accepts 100-megapixel images (boundary off-by-one) and fully decodes them into memory The preview feature caps image dimensions but the check is off by one, so a picture that is exactly 10000x10000 (100 million pixels) is accepted and fully loaded into memory. A user can request several such previews at once and consume large amounts of server memory. | filebrowser | preview-thumbnails | medium |
| medium Possibly Valid | Public directory-share access rules evaluated in wrong path namespace after re-root When someone shares a folder with a public link, the app is supposed to keep honoring any "block access to this subfolder" rules the owner set. Because the code changes how paths are named once the share opens, those block rules stop matching, so a visitor could reach a subfolder the owner intended to keep private. | filebrowser | raw-download-archive | medium |
| medium Possibly Valid | Preview opens file descriptor before admission and waits on an uncancelable context, pinning fds/goroutines When generating an image preview, the app opens the file first and then waits for a free worker slot using a wait that ignores the client hanging up. Pending or abandoned requests keep holding open files and background tasks that nothing releases, so many at once can exhaust server resources. | filebrowser | preview-thumbnails | medium |
| medium Possibly Valid | No server-side session invalidation on password change/logout; JWTs stay valid and can be renewed indefinitely Changing your password or logging out does not actually cut off sessions that are already signed in. An old login token keeps working until it expires, and it can be repeatedly refreshed to stay alive, so someone who got hold of a token is not locked out by a password change. | filebrowser | authn-jwt-session | medium |
| medium Possibly Valid | Unauthenticated login/signup decode an unbounded, un-timed request body before any credential check Anyone on the internet can hit the login and signup pages without an account, and the server reads their entire request into memory before checking any password. There is no limit on how big or how slow that request can be, so an unauthenticated attacker can push the server to consume excessive memory or hold… | filebrowser | auth-methods | medium |
| medium Possibly Valid | tus PATCH upload copy has no read/idle deadline or concurrency cap, enabling slow-body resource exhaustion When a logged-in user uploads a file in chunks, the server keeps the connection, an open file, and a background helper alive for as long as the upload lasts, with no time limit and no cap on how many can run at once. A user who opens many deliberately slow uploads can tie up the server's resources. | filebrowser | tus-upload | medium |
| medium Possibly Valid | Public share auth treats an empty stored Token as a valid match, bypassing the password on legacy tokenless shares A password-protected share whose stored access token is empty can be opened without the password by supplying an empty token in the URL. This affects older shares created before the token feature existed, letting anyone with the share link view protected content without knowing the password. | filebrowser | shares-public | medium |
| medium Possibly Valid | WebSocket /api/command upgrades and reads unbounded/undeadlined before the exec-authorization check Any logged-in user can open the command WebSocket even without permission to run commands, and the server will hold the connection open forever or accept an unlimited-size message before it checks their permission. Repeating this can tie up server memory and connections until the service degrades. | filebrowser | command-execution | high |
| medium Possibly Valid | Path deny rules match case-sensitively while the OS filesystem resolves case-insensitively, allowing rule bypass Access rules that block a folder or file compare names letter-for-letter, but on Mac, Windows, and some network drives the filesystem treats different capitalizations as the same file. So a blocked path like /secret can be reached by asking for /Secret. | filebrowser | filesystem-path-resolution | medium |
| medium Possibly Valid | Redis upload cache has no orphan-file reclamation, unlike the memory backend, leaking abandoned tus uploads on disk When file-uploads are configured to use Redis (for multi-server setups), abandoned or incomplete uploads are never cleaned off disk. The memory-based setup deletes them automatically, but the Redis one has no such cleanup, so leftover files pile up until the disk fills. | filebrowser | tus-upload | high |
| medium Possibly Valid | Hook re-authentication silently erases admin-configured per-user access Rules When an external login hook is used, logging back in can quietly wipe the custom access restrictions an administrator set on that user account. A rule meant to block the user from certain files can silently disappear, giving the user more access than intended. | filebrowser | auth-methods | high |
| medium Possibly Valid | File-event hook subprocesses spawn without concurrency, timeout, or Perm.Execute gating (resource abuse) When the administrator has set up commands that run automatically on file changes, any user allowed to create or edit files can trigger those commands over and over with no limit. Rapid file operations can pile up unlimited background processes and slow down or crash the server. | filebrowser | command-execution | medium |
| medium Possibly Valid | No rate limiting/lockout on unauthenticated share password bcrypt check (brute-force + CPU-exhaustion) For password-protected public share links, anyone who has the link can try passwords as fast as the server can check them, because there is no limit, delay, or lockout on failed attempts. This allows both password guessing and turning cheap requests into heavy server CPU load. | filebrowser | shares-public | medium |
| low Fully Valid | SPA index document is served with no Content-Security-Policy because the mux CSP middleware never runs for the NotFoundHandler The main application page is delivered without the Content-Security-Policy protection the app intends to apply. The policy is added by middleware that only runs for matched routes, but the main page is served through the router's not-found handler, which that middleware never touches. | filebrowser | http-router-middleware | high |
| low Fully Valid | MakeUserDir creates user home directories with 0777 instead of the configured DirMode (0750) New user folders are created with the most permissive access setting instead of the stricter one the application is configured to use. On a shared machine, other local users could read or change those folders. | filebrowser | settings-config | high |
| low Fully Valid | Directory delete removes share links by separator-unaware, cross-user path prefix, deleting other users' share records When a user deletes a folder, the app also deletes share links whose stored path merely starts with the same text — including unrelated folders and links belonging to other users. Those users' share URLs silently stop working. | filebrowser | storage-persistence | high |
| low Fully Valid | Malformed credential-bearing Redis URL is echoed verbatim into startup error output, leaking user:password (CWE-532/CWE-209) If the Redis connection string used for upload caching is misspelled in a way that makes it fail basic URL parsing, the server prints the whole string — including the username and password — into its startup output and logs. | filebrowser | tus-upload | high |
| low Fully Valid | Subtitle auto-detection bypasses the access-rules checker, disclosing hidden/deny-ruled file paths When the app lists a folder that contains a video, it automatically looks for matching subtitle files and returns their names — even for files that the folder's own visibility rules were supposed to hide. Someone browsing the folder can therefore learn that hidden files exist and see their names. | filebrowser | filesystem-path-resolution | high |
| low Fully Valid | No X-Frame-Options or CSP frame-ancestors on any response leaves the authenticated UI fully framable (clickjacking) The application never tells browsers it should not be embedded in a frame. A malicious site can load the logged-in interface inside a hidden frame and trick a user into clicking controls that perform real file actions without realizing it. | filebrowser | http-router-middleware | high |
| low Fully Valid | Request path logged verbatim allows CRLF log forging and terminal-escape injection (CWE-117) The server writes the requested URL straight into its log without cleaning it up. Because a URL can contain hidden newline characters, anyone sending requests can inject fake lines into the log or embed control characters that mislead whoever reads it. | filebrowser | http-router-middleware | high |
| low Fully Valid | File checksum endpoint omits the Perm.Download gate its sibling branch enforces, giving a content oracle to no-download users Users who are explicitly denied the ability to download files can still ask the server for a file's cryptographic hash. That lets them confirm whether a file has specific known contents or detect when it changes, even though they were never supposed to read the file. | filebrowser | resource-file-crud | high |
| low Fully Valid | Public authentication endpoints have no rate limiting, lockout, or anti-automation (CWE-307) The login and related sign-in endpoints accept unlimited attempts with no throttling, temporary lockout, or bot protection. An attacker can automate high-volume guessing against accounts, limited only by how long each password check takes. | filebrowser | http-router-middleware | medium |
| low Fully Valid | Public share-password verification has no rate limiting and no minimum password length, enabling online brute force Password-protected share links can be guessed at without limit: the server never throttles or locks out repeated wrong guesses, and it never enforces a minimum length for share passwords. A weak share password can be brute-forced online by anyone holding the share link. | filebrowser | shares-public | medium |
| low Fully Valid | Adversarial SRT timestamp line triggers unrecovered index-out-of-range panic in astisub parser A specially crafted subtitle file can crash the request that converts it. When a user requests conversion of such a file, the parser hits a malformed timestamp line and aborts that one connection. The damage is limited to the single failed request. | filebrowser | subtitle-conversion | high |
| low Fully Valid | Log forging via unneutralized CRLF in raw.go getFiles/rawDirHandler error logging When a file request fails, the server writes the requested file name straight into its log. An attacker can put line breaks in that name, letting them inject fake lines into the log file. This works even with no login when a public folder share is enabled. | filebrowser | shares-public | high |
| low Fully Valid | Unescaped attacker-controlled path logged with %s in raw download handler enables log injection (CWE-117) A file download URL lets a caller slip newline characters into the requested path. When the download fails, the server writes that path straight into its log without cleaning it, so an attacker can add fake lines to the log. On a password-less public share this works without even logging in. | filebrowser | raw-download-archive | high |
| low Fully Valid | tus upload memory-cache eviction logs unescaped host path enabling out-of-band log injection (CWE-117) A user uploading a file can put newline characters into the upload path. If the upload is abandoned, a background cleanup timer later writes that path into the log without escaping it, letting the user inject fake log lines. It only needs upload permission and works on the default setup. | filebrowser | tus-upload | medium |
| low Possibly Valid | reCAPTCHA verification uses an http.Client with no timeout on the synchronous login path (login-thread stall risk) When reCAPTCHA is enabled, each login waits for a reply from the reCAPTCHA service using a connection that has no time limit. If that service is slow or unreachable, login requests can hang indefinitely and stack up. | filebrowser | auth-methods | medium |
| low Possibly Valid | PATCH copy/rename runs before_copy/before_rename hooks before the Perm.Create/Perm.Rename check, letting unpermitted users trigger hook commands For copy and rename requests, the server runs the admin-configured 'before' hook command first and only afterward checks whether the user is allowed to copy or rename. A user lacking those permissions can still make the hook run with a filename and destination they control. | filebrowser | resource-file-crud | medium |
| low Possibly Valid | Subtitle conversion endpoint fully parses and buffers files in memory with no pre-read size cap (resource-exhaustion DoS) When someone requests a subtitle file, the server reads the whole file into memory and builds the converted output entirely in memory before sending anything, with no limit on file size. Large or crafted subtitle files, requested repeatedly, can drive up memory use and strain the server. | filebrowser | subtitle-conversion | medium |
| low Possibly Valid | Proxy auth trusts an unvalidated header and auto-provisions an empty-username account when the header is absent Under proxy authentication the app takes the username straight from a request header without checking it is present or non-blank. If the header is missing, it can auto-create a blank-named account with default permissions and hand out a login token for it. | filebrowser | auth-methods | medium |
| low Possibly Valid | Preview serves embedded EXIF thumbnail bytes verbatim without the nosniff/script-src hardening siblings apply When making a small preview of a JPEG, the app can send back a thumbnail that is stored inside the uploaded file, exactly as the uploader wrote it, without the safety headers that similar download endpoints add. On an unusually configured server this raw content could be misinterpreted by the browser as a web page. | filebrowser | preview-thumbnails | low |
| low Possibly Valid | Recursive listing handler omits the root-path authorization Check The recursive folder-listing feature skips the access check on the folder you point it at, unlike every other file operation. In certain rule configurations this lets a user list the names, sizes, and dates of items inside a folder they were supposed to be blocked from. | filebrowser | resource-file-crud | medium |
| low Possibly Valid | PATCH resource handler fires client-selected before_<action> hooks before any per-action permission check When a user sends a file-management request, the server reads the requested action straight from the URL and runs the matching admin-configured 'before' command first, before checking whether that user is even allowed to do it. A user can trigger commands they were never permitted to run. | filebrowser | command-execution | medium |
| low Possibly Valid | Login reCAPTCHA verification checks only the success flag, allowing cross-origin token replay When the optional login CAPTCHA is turned on, the server only trusts that Google said the puzzle was solved, but never checks that it was solved on the real site. An attacker could collect solved puzzles from a page they control and reuse them to get past the CAPTCHA on the login form. | filebrowser | auth-methods | low |
| low Possibly Valid | Discarded afero WalkFunc error yields nil FileInfo dereference (nil-pointer panic) in search handler The search feature ignores errors while walking the file tree. If a file disappears at the wrong moment during a search, the code uses an empty file record and crashes that one search request. Only the single request fails; the server keeps running. | filebrowser | search | medium |
| low Possibly Valid | Password-equivalent share token carried in URL query string with no Referrer-Policy (CWE-598) A password-protected share can be opened with a secret token instead of the password, and that token travels in the web address. Web addresses commonly end up in server logs, browser history, and forwarded links, so anyone who sees the address can reopen the share without knowing the password. | filebrowser | shares-public | medium |
| low Possibly Valid | marshal() backup write follows symlinks on predictable users.backup.json path (CWE-59 confused-deputy overwrite) When importing users with the --replace option, the tool writes a backup to a fixed, predictable filename in the current directory using an open call that follows symbolic links. If someone pre-plants a symlink there pointing at another file, running the command overwrites and destroys that other file. | filebrowser | cli-bootstrap | medium |
| low Possibly Valid | WebSocket /api/command lacks timeout and kill-on-disconnect, orphaning child processes (availability/resource abuse) When a user runs a command over the WebSocket endpoint, the server does not tie the command's lifetime to the connection. If the user disconnects, the command keeps running and its goroutine stays parked, so repeatedly starting and dropping long-running commands can pile up abandoned processes. | filebrowser | command-execution | medium |
| low Possibly Valid | Unbounded in-memory buffering during TIFF format detection in image preview When the server makes a thumbnail of a TIFF image, it can be tricked into loading a huge amount of the file into memory before any size limit applies. A user with a very large crafted TIFF can spike server memory and, with a few concurrent requests, risk crashing it. | filebrowser | preview-thumbnails | medium |
| low Possibly Valid | PATCH /api/resources action parameter unvalidated, firing unauthorized before_/after_ hooks (missing authorization/input allowlist) The file-move endpoint accepts an 'action' name from the URL without checking it against a list of allowed actions. That name is used to fire configured shell hooks, so a user with no permissions can trigger hooks for operations they are not allowed to perform, such as a delete hook. | filebrowser | resource-file-crud | medium |
| low Possibly Valid | Username-enumeration timing side channel in hook 'pass' auth action (missing dummyHash equalization) When the app is configured to check passwords through an external hook, a login attempt for a real username takes measurably longer than one for a username that does not exist. An attacker timing these responses could learn which usernames are valid, even without guessing any password. | filebrowser | auth-methods | low |
| low Possibly Valid | Unauthenticated public download endpoint lacks WriteTimeout, enabling slow-read resource holding Anyone can download files from password-less public share links without logging in, and the server has no time limit on how long a download response may take. A client that reads very slowly can tie up server connections and memory, hurting availability. | filebrowser | raw-download-archive | low |