From: "Aurélien Aptel" <aaptel@suse.com>
To: Ronnie Sahlberg <lsahlber@redhat.com>,
linux-cifs <linux-cifs@vger.kernel.org>
Cc: Steve French <smfrench@gmail.com>,
Paulo Alcantara <paulo@paulo.ac>,
Ronnie Sahlberg <lsahlber@redhat.com>
Subject: Re: [PATCH] cifs: fix crash in cifs_dfs_do_automount
Date: Fri, 12 Jul 2019 09:56:41 +0200 [thread overview]
Message-ID: <87lfx34reu.fsf@suse.com> (raw)
In-Reply-To: <20190711034658.21485-1-lsahlber@redhat.com>
Hi Ronnie,
Ronnie Sahlberg <lsahlber@redhat.com> writes:
> RHBZ: 1649907
>
> Fix a crash that happens while attempting to mount a DFS referral from the same server on the root of a filesystem.
>
> Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>
> ---
> fs/cifs/connect.c | 15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
> index 8ad8bbe8003b..9b0f9f346c5b 100644
> --- a/fs/cifs/connect.c
> +++ b/fs/cifs/connect.c
> @@ -4484,11 +4484,13 @@ cifs_are_all_path_components_accessible(struct TCP_Server_Info *server,
> unsigned int xid,
> struct cifs_tcon *tcon,
> struct cifs_sb_info *cifs_sb,
> - char *full_path)
> + char *full_path,
> + int added_treename)
> {
> int rc;
> char *s;
> char sep, tmp;
> + int skip = added_treename ? 1 : 0;
>
> sep = CIFS_DIR_SEP(cifs_sb);
> s = full_path;
> @@ -4503,7 +4505,13 @@ cifs_are_all_path_components_accessible(struct TCP_Server_Info *server,
> /* next separator */
> while (*s && *s != sep)
> s++;
> -
> + /* if the treename is added, we then have to skip the first
> + * part within the separators
> + */
Nitpicking (Steve can probably fix this when he applies) but comment
style should be
/*
* foo
*/
> + if (skip) {
> + skip = 0;
> + continue;
> + }
> /*
> * temporarily null-terminate the path at the end of
> * the current component
> @@ -4551,8 +4559,7 @@ static int is_path_remote(struct cifs_sb_info *cifs_sb, struct smb_vol *vol,
>
> if (rc != -EREMOTE) {
> rc = cifs_are_all_path_components_accessible(server, xid, tcon,
> - cifs_sb,
> - full_path);
> + cifs_sb, full_path, tcon->Flags & SMB_SHARE_IS_IN_DFS);
Just FYI this flag is just set in SMB1. Can we test this change in the buildbot?
> if (rc != 0) {
> cifs_dbg(VFS, "cannot query dirs between root and final path, "
> "enabling CIFS_MOUNT_USE_PREFIX_PATH\n");
> --
> 2.13.6
>
>
Cheers,
--
Aurélien Aptel / SUSE Labs Samba Team
GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3
SUSE Linux GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 21284 (AG Nürnberg)
next prev parent reply other threads:[~2019-07-12 7:56 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-11 3:46 [PATCH] cifs: fix crash in cifs_dfs_do_automount Ronnie Sahlberg
2019-07-12 7:56 ` Aurélien Aptel [this message]
2019-07-13 17:10 ` Steve French
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87lfx34reu.fsf@suse.com \
--to=aaptel@suse.com \
--cc=linux-cifs@vger.kernel.org \
--cc=lsahlber@redhat.com \
--cc=paulo@paulo.ac \
--cc=smfrench@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).