Calibre cover path check only lstats the final path component #28

Open
opened 2026-10-18 16:23:37 +00:00 by jfro · 0 comments
Owner

Non-blocking follow-up from Nye's approval of #18 (fixes #5).

store_synced_cover checks lstat on the final path component only, so a symlinked directory inside the Calibre library could still point outside it.

Fix: resolve the real path (walk components with :file.read_link_info, or compare against the realpath of the library root). The library is admin-controlled, so low.

Non-blocking follow-up from Nye's approval of #18 (fixes #5). `store_synced_cover` checks `lstat` on the final path component only, so a symlinked *directory* inside the Calibre library could still point outside it. **Fix:** resolve the real path (walk components with `:file.read_link_info`, or compare against the realpath of the library root). The library is admin-controlled, so low.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
jfro/fuzzy_catalog#28
No description provided.