GNU bug report logs - #3230
dired-actual-switches is risky

Previous Next

Package: emacs;

Reported by: Leo <sdl.web <at> gmail.com>

Date: Wed, 6 May 2009 14:25:06 UTC

Severity: normal

Fixed in version 24.1

Done: Glenn Morris <rgm <at> gnu.org>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Glenn Morris <rgm <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#3230: closed (dired-actual-switches is risky)
Date: Tue, 01 Mar 2011 03:27:02 +0000

[Message part 1 (text/plain, inline)]
Your message dated Mon, 28 Feb 2011 22:25:59 -0500
with message-id <krpqqbttbs.fsf <at> fencepost.gnu.org>
and subject line Re: bug#3230: 23.0.93; Make dired-actual-switches safe local variable?
has caused the GNU bug report #3230,
regarding dired-actual-switches is risky
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
3230: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3230
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]
From: Leo <sdl.web <at> gmail.com>
To: emacs-pretest-bug <at> gnu.org
Subject: 23.0.93; Make dired-actual-switches safe local variable?
Date: Wed, 06 May 2009 15:20:57 +0100

Hi there,

The dired-x manual gives an example in using local variables for dired
buffers. However, the variable dired-actual-switches has not been marked
as safe local variable. I think this is an oversight.

,----[ (info "(dired-x)Local Variables") ]
| For example, if the user puts
| 
|      Local Variables:
|      dired-actual-switches: "-lat"
|      dired-omit-mode: t
|      End:
`----

Best wishes,

Leo



[Message part 3 (message/rfc822, inline)]
From: Glenn Morris <rgm <at> gnu.org>
To: 3230-done <at> debbugs.gnu.org
Subject: Re: bug#3230: 23.0.93; Make dired-actual-switches safe local variable?
Date: Mon, 28 Feb 2011 22:25:59 -0500

Version: 24.1

Stefan Monnier wrote:

> should then go to safe-local-variable.  I'd recommend something simple
> like
>
>   (defun dired-safe-switches-p (switches)
>     (string-match "\\`[- [[:alnum:]]]+\\'" switches))

Added.
This bug report was last modified 14 years and 82 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.