Missing error handling for invalid lock file content #4

New Issue

snegov · 2025-11-15T03:44:09Z

snegov commented

2025-11-15 03:44:09 +00:00

Missing error handling for invalid lock file content

Priority: Medium
Component: backup.py
Type: Bug

Description

The lock file parsing in set_backups_lock() assumes the file contains a valid integer, but there's no error handling if the file is corrupted or contains non-numeric data.

Location

curateipsum/backup.py:114-115

Current Code

with open(lock_file_path, "r") as f:
    pid = int(f.read())

Problem

If the lock file is corrupted or empty, int() will raise a ValueError, causing the backup to crash rather than handling the situation gracefully.

Proposed Solution

try:
    with open(lock_file_path, "r") as f:
        pid = int(f.read().strip())
except (ValueError, IOError) as e:
    _lg.warning("Corrupted lock file, removing and continuing: %s", e)
    os.unlink(lock_file_path)
    return set_backups_lock(backups_dir, force)

Impact

Medium - Can cause backup process to crash if lock file is corrupted.

# Missing error handling for invalid lock file content **Priority:** Medium **Component:** backup.py **Type:** Bug ## Description The lock file parsing in `set_backups_lock()` assumes the file contains a valid integer, but there's no error handling if the file is corrupted or contains non-numeric data. ## Location `curateipsum/backup.py:114-115` ## Current Code ```python with open(lock_file_path, "r") as f: pid = int(f.read()) ``` ## Problem If the lock file is corrupted or empty, `int()` will raise a `ValueError`, causing the backup to crash rather than handling the situation gracefully. ## Proposed Solution ```python try: with open(lock_file_path, "r") as f: pid = int(f.read().strip()) except (ValueError, IOError) as e: _lg.warning("Corrupted lock file, removing and continuing: %s", e) os.unlink(lock_file_path) return set_backups_lock(backups_dir, force) ``` ## Impact **Medium** - Can cause backup process to crash if lock file is corrupted.

snegov referenced this issue from a commit

2026-02-04 06:07:44 +00:00

Fix high-priority bugs and add comprehensive test coverage

snegov referenced this issue from a commit

2026-02-04 06:09:54 +00:00

Fix high-priority bugs and add comprehensive test coverage

snegov closed this issue

2026-02-04 06:09:55 +00:00

Sign in to join this conversation.