mirror of
https://github.com/gogs/gogs.git
synced 2026-01-09 08:52:39 +01:00
pkg/tool: improve SanitizePath (#5558)
This commit is contained in:
@@ -17,5 +17,7 @@ func IsSameSiteURLPath(url string) bool {
|
||||
|
||||
// SanitizePath sanitizes user-defined file paths to prevent remote code execution.
|
||||
func SanitizePath(path string) string {
|
||||
return strings.TrimLeft(path, "./")
|
||||
path = strings.TrimLeft(path, "/")
|
||||
path = strings.Replace(path, "../", "", -1)
|
||||
return path
|
||||
}
|
||||
|
||||
@@ -38,6 +38,7 @@ func Test_SanitizePath(t *testing.T) {
|
||||
expect string
|
||||
}{
|
||||
{"../../../../../../../../../data/gogs/data/sessions/a/9/a9f0ab6c3ef63dd8", "data/gogs/data/sessions/a/9/a9f0ab6c3ef63dd8"},
|
||||
{"data/gogs/../../../../../../../../../data/sessions/a/9/a9f0ab6c3ef63dd8", "data/gogs/data/sessions/a/9/a9f0ab6c3ef63dd8"},
|
||||
|
||||
{"data/sessions/a/9/a9f0ab6c3ef63dd8", "data/sessions/a/9/a9f0ab6c3ef63dd8"},
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user