From 708ad138c8760e9e601adbd5d9f71c653e6a5602 Mon Sep 17 00:00:00 2001 From: Henrik Lissner Date: Sat, 1 Sep 2018 00:02:39 +0200 Subject: [PATCH] doom/sudo-find-file: don't sudo writeable files --- core/autoload/files.el | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/core/autoload/files.el b/core/autoload/files.el index 9afcb041d..99831229b 100644 --- a/core/autoload/files.el +++ b/core/autoload/files.el @@ -5,11 +5,11 @@ "Open FILE as root." (interactive (list (read-file-name "Open as root: "))) - (find-file (if (file-writable-p file) - file - (if (file-remote-p file) - (concat "/" (file-remote-p file 'method) ":" (file-remote-p file 'user) "@" (file-remote-p file 'host) "|sudo:root@" (file-remote-p file 'host) ":" (file-remote-p file 'localname)) - (concat "/sudo:root@localhost:" file))))) + (when (file-writable-p file) + (user-error "File is user writeable, aborting sudo")) + (find-file (if (file-remote-p file) + (concat "/" (file-remote-p file 'method) ":" (file-remote-p file 'user) "@" (file-remote-p file 'host) "|sudo:root@" (file-remote-p file 'host) ":" (file-remote-p file 'localname)) + (concat "/sudo:root@localhost:" file)))) ;;;###autoload (defun doom/sudo-this-file ()