Skip to content

Commit 5ad623e

Browse files
committed
Fix possible server-size DoS vulnerability
Signed-off-by: Kirill K. Smirnov <[email protected]>
1 parent 2002823 commit 5ad623e

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/flask_se_auth.py

+6
Original file line numberDiff line numberDiff line change
@@ -248,6 +248,12 @@ def upload_avatar():
248248
if file.filename == "":
249249
flash("No selected file")
250250
return redirect(request.url)
251+
# Sanity check: limit uploadable filename
252+
# to avoid excessive burden to NFKD normalization
253+
# in secure_filename() method
254+
if len(file.filename) > 1000:
255+
flash("Filename too long")
256+
return redirect(request.url)
251257
if file and allowed_file(file.filename):
252258
filename = secure_filename(file.filename)
253259
new_filename = os.urandom(16).hex()

0 commit comments

Comments
 (0)