{"record":{"id":"826e16bf44c204db","repo":"Tencent/WeKnora","slug":"invalid-filename-contains-path-traversal","errorCode":null,"errorMessage":"invalid fileName: contains path traversal","messagePattern":"invalid fileName: contains path traversal","errorType":"validation","errorClass":null,"httpStatus":null,"severity":"error","filePath":"internal/utils/security.go","lineNumber":139,"sourceCode":"\tsep := string(filepath.Separator)\n\tif absPath != absBase && !strings.HasPrefix(absPath, absBase+sep) {\n\t\treturn \"\", fmt.Errorf(\"path traversal denied: path is outside base directory\")\n\t}\n\treturn absPath, nil\n}\n\n// SafeFileName 校验并返回安全的“仅文件名”部分，防止路径遍历。\n// 仅保留最后一个路径成分，禁止 \"..\"、空名或仅含点，用于 SaveBytes 等场景。\nfunc SafeFileName(fileName string) (string, error) {\n\tif fileName == \"\" {\n\t\treturn \"\", fmt.Errorf(\"fileName cannot be empty\")\n\t}\n\tbase := filepath.Base(filepath.Clean(fileName))\n\tif base == \"\" || base == \".\" || base == \"..\" {\n\t\treturn \"\", fmt.Errorf(\"invalid fileName: path traversal or empty name\")\n\t}\n\tif strings.Contains(base, \"..\") {\n\t\treturn \"\", fmt.Errorf(\"invalid fileName: contains path traversal\")\n\t}\n\tif len(base) > 255 {\n\t\treturn \"\", fmt.Errorf(\"fileName too long\")\n\t}\n\treturn base, nil\n}\n\n// SafeObjectKey 校验对象存储的 key（如 COS/MinIO objectName），禁止包含 \"..\" 等路径遍历\nfunc SafeObjectKey(objectKey string) error {\n\tif objectKey == \"\" {\n\t\treturn fmt.Errorf(\"object key cannot be empty\")\n\t}\n\tif strings.Contains(objectKey, \"..\") {\n\t\treturn fmt.Errorf(\"object key contains path traversal\")\n\t}\n\treturn nil\n}\n","sourceCodeStart":121,"sourceCodeEnd":157,"githubUrl":"https://github.com/Tencent/WeKnora/blob/988cbb03305e055d8ebb7d46d9ac6cc0803cd074/internal/utils/security.go#L121-L157","documentation":"Traversal validation in SafeFileName: after reducing the input with Base/Clean, the result still contains '..' anywhere, meaning the value embeds a traversal sequence that survived the first checks. The function rejects it outright so only a plain final path component can ever be used for file writes.","triggerScenarios":"Thrown at internal/utils/security.go:139 when the library encounters an invalid state.","commonSituations":"See trigger scenarios.","solutions":["Sanitize or replace '..' sequences in user-supplied names","Reject the upload and log the traversal attempt","Generate a server-side safe name instead of trusting the client name"],"exampleFix":null,"handlingStrategy":"validation","validationCode":null,"typeGuard":null,"tryCatchPattern":null,"preventionTips":[],"tags":[],"backgroundTag":null,"analyzedSha":"988cbb03305e055d8ebb7d46d9ac6cc0803cd074","analyzedAt":"2026-09-02T14:41:08.344Z","contentChangedAt":null,"schemaVersion":2},"datasetVersion":"2026-09-08T10:18:20.063Z"}