Commit 564d36ca authored by Eric W. Biederman's avatar Eric W. Biederman Committed by Kleber Sacilotto de Souza

fs/nfs: Fix nfs_parse_devname to not modify it's argument

BugLink: https://bugs.launchpad.net/bugs/1864773

[ Upstream commit 40cc394b ]

In the rare and unsupported case of a hostname list nfs_parse_devname
will modify dev_name.  There is no need to modify dev_name as the all
that is being computed is the length of the hostname, so the computed
length can just be shorted.

Fixes: dc045898 ("NFS: Use common device name parsing logic for NFSv4 and NFSv2/v3")
Signed-off-by: default avatar"Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: default avatarTrond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarKhalid Elmously <khalid.elmously@canonical.com>
Signed-off-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
parent a8009a59
...@@ -1901,7 +1901,7 @@ static int nfs_parse_devname(const char *dev_name, ...@@ -1901,7 +1901,7 @@ static int nfs_parse_devname(const char *dev_name,
/* kill possible hostname list: not supported */ /* kill possible hostname list: not supported */
comma = strchr(dev_name, ','); comma = strchr(dev_name, ',');
if (comma != NULL && comma < end) if (comma != NULL && comma < end)
*comma = 0; len = comma - dev_name;
} }
if (len > maxnamlen) if (len > maxnamlen)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment