--- Begin Message ---
|
Subject: |
give better message when trying to go to forbidden areas |
|
Date: |
Wed, 11 Jan 2017 22:44:08 +0800 |
Do C-x C-f ~root
You will see:
Setting current directory: Success, /root/
But that is about all.
Because normally,
$ cd ~root
bash: cd: /root: Permission denied
Which better reflects reality.
--- End Message ---
--- Begin Message ---
|
Subject: |
Re: bug#25419: give better message when trying to go to forbidden areas |
|
Date: |
Fri, 27 Jan 2017 10:59:08 +0200 |
> From: address@hidden
> Cc: 積丹尼 Dan Jacobson <address@hidden>,
> address@hidden
> Date: Wed, 18 Jan 2017 22:28:17 -0500
>
> Eli Zaretskii <address@hidden> writes:
>
> >> From: 積丹尼 Dan Jacobson
> >> <address@hidden>
> >> Date: Wed, 11 Jan 2017 22:44:08 +0800
> >>
> >> Do C-x C-f ~root
> >> You will see:
> >> Setting current directory: Success, /root/
> >
> > That's because /root has a file-name handler,
> > tramp-completion-file-name-handler, and although that handler turns
> > around and calls file-accessible-directory-p, the value of errno is
> > lost in transition:
> >
> > /* If the file name has special constructs in it,
> > call the corresponding file handler. */
> > handler = Ffind_file_name_handler (absname, Qfile_accessible_directory_p);
> > if (!NILP (handler))
> > {
> > Lisp_Object r = call2 (handler, Qfile_accessible_directory_p,
> > absname);
> > errno = 0;
> > return r;
> > }
> >
> > Is there any better way than arbitrarily set errno to EACCES if the
> > handler returns something other than t?
>
> AFAICT, not without changing the interface of
> file-accessible-directory-p file handler, no.
OK, thanks. I made that change, and I'm marking this bug done.
--- End Message ---