From: Patrick Farrell <pfarrell@whamcloud.com>
Date: Fri, 11 Oct 2019 01:36:13 +0000 (-0400)
Subject: LU-12726 mdt: Fix usage of sscanf
X-Git-Tag: 2.13.51~200
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=8a0a6dbec5537764958d4863e76de249fdd675a4;p=fs%2Flustre-release.git

LU-12726 mdt: Fix usage of sscanf

sscanf is returning the number of items matched in the
input, but we need to return the amount of data
successfully written.

Fixes: a408e9dd426f ("LU-8066 mdt: migrate procfs files to sysfs")

Signed-off-by: Patrick Farrell <pfarrell@whamcloud.com>
Change-Id: Ib90bd260e745692d11656d0b74820573aaa35550
Reviewed-on: https://review.whamcloud.com/36052
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---

diff --git a/lustre/mdt/mdt_lproc.c b/lustre/mdt/mdt_lproc.c
index a0cf377..1ad735e 100644
--- a/lustre/mdt/mdt_lproc.c
+++ b/lustre/mdt/mdt_lproc.c
@@ -303,7 +303,7 @@ static ssize_t identity_upcall_store(struct kobject *kobj,
 
 	/* Remove any extraneous bits from the upcall (e.g. linefeeds) */
 	down_write(&hash->uc_upcall_rwsem);
-	count = sscanf(buffer, "%s", hash->uc_upcall);
+	sscanf(buffer, "%s", hash->uc_upcall);
 	up_write(&hash->uc_upcall_rwsem);
 
 	if (strcmp(hash->uc_name, mdt_obd_name(mdt)) != 0)