SiYuan: getUniqueFilename passes an unvalidated client-supplied path to the filesystem, giving anonymous readers an existence oracle over the entire host filesystem

QFinch assessment

/api/file/getUniqueFilename takes a path from the request body and passes it to a filesystem existence check with no validation, confinement or authorization. The response distinguishes paths that exist from paths that do not, so an anonymous reader in publish mode can probe arbitrary locations on the host, one request per probe. Files and directories both work. Every neighbouring file route either requires an administrator or confines the path first.

Potential impact

An anonymous reader in publish mode, or any publish RoleReader, determines whether arbitrary paths exist on the host. Because directories respond the same way as files, the filesystem can be walked rather than only guessed at. This supports identifying the operating-system user's home directory, detecting installed software, backup and synchronisation directories, mapping the workspace layout, and confirming the existence of specific documents by probing their storage paths. It also composes with the absolute workspace path returned elsewhere, which supplies a concrete prefix rather than requiring guesswork, and it recovers the operating-system username that the browser-request path strip added under issue 17410 was introduced to conceal. Scoped precisely: nothing is read and nothing is written. This is an existence and layout disclosure, valuable primarily as a reconnaissance step.

Technical context

Route. kernel/api/router.go:289 on master, :292 on the development branch, CheckAuth only. Handler, kernel/api/file.go:84, in full: The argument binder does no path handling. util.BindJsonArg (kernel/util/net.go:365) takes two booleans, required (the key must be present) and rejectEmpty (the value must be non-empty). That is its entire contract. There is no filepath.Clean, no workspace join, no traversal check. An absolute path outside the workspace reaches gulu.File.IsExist unmodified. The oracle. util/file.go:67: Input echoed back means the path does not exist. A (1) suffix means it does. The asymmetry. | Route | Middleware | |---|---| | putFile, copyFile, globalCopyFiles, workspaceCopyFiles, removeFile, renameFile | CheckAuth, CheckAdminRole, CheckReadonly | | readDir (directory enumeration) | CheckAuth, CheckAdminRole | | getFile | CheckAuth, but GetAbsPathInWorkspace returns 403 on escape, plus rejectEncryptedBoxPath and a publish-access check | | getUniqueFilename | CheckAuth, nothing else | getFile binds the identical BindJsonArg("path", &filePath, true, true) and confines it as its first action.