summaryrefslogtreecommitdiff
path: root/modules/selection-framework.el
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-23 03:31:04 -0500
committerCraig Jennings <c@cjennings.net>2026-05-23 03:31:04 -0500
commit04ce04c898851cfa49dac2d1dec2dff17465e87d (patch)
tree67daf626fa05a8efdec8c7f16aa0833dba46925c /modules/selection-framework.el
parent9791a7c0855c90d74b0ff5c33b03a0ccd816cb4a (diff)
downloaddotemacs-04ce04c898851cfa49dac2d1dec2dff17465e87d.tar.gz
dotemacs-04ce04c898851cfa49dac2d1dec2dff17465e87d.zip
fix(dirvish): guard nil file and reject path-traversal playlist names
cj/set-wallpaper passed `(dired-file-name-at-point)` straight to `expand-file-name`, so running it with no file at point raised a bare `wrong-type-argument` instead of a clear error. cj/dired-create-playlist-from-marked expanded the raw playlist name under `music-dir` without checking it, so a name like "../foo" or "/etc/foo" would write outside the music directory. I added a nil-file guard to set-wallpaper and a `cj/--playlist-name-safe-p` check that rejects any name carrying a directory separator before the path is built. Both paths now fail cleanly with a user-error. Regression tests went into the existing wrapper and playlist test files.
Diffstat (limited to 'modules/selection-framework.el')
0 files changed, 0 insertions, 0 deletions