[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#16984: dired-do-rename susceptible to .../~/... hijack
From: |
Michael Albinus |
Subject: |
bug#16984: dired-do-rename susceptible to .../~/... hijack |
Date: |
Thu, 08 Dec 2016 15:58:43 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
npostavs@users.sourceforge.net writes:
Hi Noam,
> (let ((local (file-remote-p filename 'localname)))
Well, this is (file-local-name filename) these days.
> Regarding `tramp-quoted-name-p', you should use (string-match "\\`/:"
> ...) or (string-prefix-p "/:" ...), not (string-match "^/:" ...) as the
> latter could get confused by newlines in filenames (of the first two, I
> find the `string-prefix-p' version clearer).
In general you are right. But we don't support newlines in file names;
at least Tramp doesn't, so it makes no difference using "^/:" or "\\`/:".
`string-prefix-p' is the best choice indeed, and I thought about, but
Tramp cannot apply it yet. It has been introduced in Emacs 24.1, and
Tramp still supports Emacs 23. Well, NEWS.24 says that it exists already
since Emacs 23.2, but there is still Emacs 23.1, which is supported by
Tramp.
Best regards, Michael.
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Michael Albinus, 2016/12/04
- bug#16984: dired-do-rename susceptible to .../~/... hijack, npostavs, 2016/12/07
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Michael Albinus, 2016/12/08
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Eli Zaretskii, 2016/12/08
- bug#16984: dired-do-rename susceptible to .../~/... hijack, npostavs, 2016/12/08
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Michael Albinus, 2016/12/09
- bug#16984: dired-do-rename susceptible to .../~/... hijack, npostavs, 2016/12/11
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Eli Zaretskii, 2016/12/09
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Eli Zaretskii, 2016/12/08
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Michael Albinus, 2016/12/08
- bug#16984: dired-do-rename susceptible to .../~/... hijack, Eli Zaretskii, 2016/12/08