Raise SecurityError on invalid path
This commit is contained in:
parent
3c46f60042
commit
75b44f6980
1 changed files with 6 additions and 2 deletions
|
@ -22,6 +22,10 @@ status_texts = {
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
class SecurityError(Exception):
|
||||||
|
pass
|
||||||
|
|
||||||
|
|
||||||
@PluginManager.acceptPlugins
|
@PluginManager.acceptPlugins
|
||||||
class UiRequest(object):
|
class UiRequest(object):
|
||||||
|
|
||||||
|
@ -417,8 +421,8 @@ class UiRequest(object):
|
||||||
if path.endswith("/"):
|
if path.endswith("/"):
|
||||||
path = path + "index.html"
|
path = path + "index.html"
|
||||||
|
|
||||||
if ".." in path:
|
if ".." in path or "./" in path:
|
||||||
raise Exception("Invalid path")
|
raise SecurityError("Invalid path")
|
||||||
|
|
||||||
match = re.match("/media/(?P<address>[A-Za-z0-9\._-]+)(?P<inner_path>/.*|$)", path)
|
match = re.match("/media/(?P<address>[A-Za-z0-9\._-]+)(?P<inner_path>/.*|$)", path)
|
||||||
if match:
|
if match:
|
||||||
|
|
Loading…
Reference in a new issue