Browse Source

routes: fix open redirect vulnerability #5364 (#5365)

pull/5380/head
chromium1337 6 years ago committed by 无闻
parent
commit
1f247cf813
  1. 4
      routes/user/auth.go

4
routes/user/auth.go

@ -73,10 +73,10 @@ func AutoLogin(c *context.Context) (bool, error) {
} }
// isValidRedirect returns false if the URL does not redirect to same site. // isValidRedirect returns false if the URL does not redirect to same site.
// False: //url, http://url // False: //url, http://url, /\url
// True: /url // True: /url
func isValidRedirect(url string) bool { func isValidRedirect(url string) bool {
return len(url) >= 2 && url[0] == '/' && url[1] != '/' return len(url) >= 2 && url[0] == '/' && url[1] != '/' && url[1] != '\\'
} }
func Login(c *context.Context) { func Login(c *context.Context) {

Loading…
Cancel
Save