Since 'snprintf()' returns the number of characters emitted, an extra call to 'strlen()' in 'ceph_mds_auth_match()' may be dropped. Compile tested only. Signed-off-by: Dmitry Antipov <dmantipov@xxxxxxxxx> --- fs/ceph/mds_client.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/ceph/mds_client.c b/fs/ceph/mds_client.c index c2157f6e0c69..7224283046a7 100644 --- a/fs/ceph/mds_client.c +++ b/fs/ceph/mds_client.c @@ -5665,9 +5665,9 @@ static int ceph_mds_auth_match(struct ceph_mds_client *mdsc, if (!_tpath) return -ENOMEM; /* remove the leading '/' */ - snprintf(_tpath, n, "%s/%s", spath + 1, tpath); + tlen = snprintf(_tpath, n, "%s/%s", + spath + 1, tpath); free_tpath = true; - tlen = strlen(_tpath); } /* -- 2.45.2