smokephil/paperless-ngx
1
0
Fork 0
mirror of https://github.com/paperless-ngx/paperless-ngx.git synced 2025-12-07 23:35:22 +01:00
Code Issues Projects Releases Packages Wiki Activity

Avoid TOCTOU error by rather catching an exception than checking

Browse source 
beforehand
This commit is contained in:
Wolf-Bastian Poettner 2020-02-01 20:13:05 +00:00
parent 6a671ebd22
commit 05da4a559f
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/documents/models.py
View file

@ -450,9 +450,9 @@ def delete_files(sender, instance, **kwargs):
# Remove the document
old_file = instance.filename_to_path(instance.filename)
if os.path.isfile(old_file):
try:
os.remove(old_file)
else:
except FileNotFoundError:
logger = logging.getLogger(__name__)
logger.warning("Deleted document " + str(instance.id) + " but file " +
old_file + " was no longer present")