Whamcloud - gitweb
LU-12726 mdt: Fix usage of sscanf 52/36052/7
authorPatrick Farrell <pfarrell@whamcloud.com>
Fri, 11 Oct 2019 01:36:13 +0000 (21:36 -0400)
committerOleg Drokin <green@whamcloud.com>
Tue, 12 Nov 2019 04:06:24 +0000 (04:06 +0000)
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>

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