From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755200AbeAMOE6 (ORCPT + 1 other); Sat, 13 Jan 2018 09:04:58 -0500 Received: from mout.web.de ([212.227.15.3]:61348 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754986AbeAMOE4 (ORCPT ); Sat, 13 Jan 2018 09:04:56 -0500 Subject: [PATCH 2/2] nfsd: Adjust four function calls together with a variable assignment From: SF Markus Elfring To: linux-nfs@vger.kernel.org, Jeff Layton , "J. Bruce Fields" Cc: LKML , kernel-janitors@vger.kernel.org References: <43df7d8b-f37e-91ca-9c8c-96e318adeb4f@users.sourceforge.net> Message-ID: <9f4ab241-a84a-0db5-6915-9e72bdbc24b0@users.sourceforge.net> Date: Sat, 13 Jan 2018 15:04:51 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 MIME-Version: 1.0 In-Reply-To: <43df7d8b-f37e-91ca-9c8c-96e318adeb4f@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:jJs/+UzAjFxH1rl96HFn5KSZQ/81v0jDtlJ74rKdzxd7ff73HCY ij3YzuJWBBgWnvDRLWkX6Pk6mnm6W/sxlcHe+SgIFLNLPtA0/CcXRmhR3fYeOJD0FNM+V2o mN/ABQgUi5YyuWpRVi9vJ0pa1Wyg7Jv2WNuEI3+DlNh1b3T2q7Al2zTGJqw2NUQW7i1/aAs Sk+HMK2QbGtw2ba0Hxnkg== X-UI-Out-Filterresults: notjunk:1;V01:K0:L5EHbTGvCus=:VflwtzxJsCizwcK2oRhm8e eqxnt3Wwr/S3v/KFdx6vlU1nXeMGnKbDdp80ouLf9AUzehDrLZlKJJIcZKrmRXvpw5/uT2VRl MHHOC7IHG6NQLY0rq0KxmMgIDyBZgNMlNNkNkizFAL3osFa/IY4fiFYzvrQb/fcaluVkp6mD6 EYcTmhOR/2CeL/ocjNrY9CdxTaSWaaSHCpkdO2EvBqit3sr5emEErvv75RNqTH86DoqncE0Lk vkpTWXbXHFg4ds08SsQ3+f+01yCHSTNl3AM1D75kter07FkuUKm5nB89IQ44o2W6ewVkh7RHm z3yMZyEDh7BEu74gBaSL/CmRBPTTcUqPeEbL4kx/1TSeVbB2HKrjW4eyKOJZd4V2eI11Dxl7v S61S0tG/c/maP7UOVSjnkOgNkQm7NlEmnZ3Hr+BNoL+jCafRZLxbLsDA8RwceHihXvM7lZBir /++qmjZJ8p7CwBJPq3jps32fJJGqGQUsqTwYztJ6wPJxS6C5hVAeujzQv6mBbgGuRMvQp5sGq Rh2Pr0sXbwQPqGx9v+husS40HSsd6xv9Qq+qjvc+jjvHiBQejskcROHi/lMIcDBT9nKqDmZtO eoC8hj1kFOuzietzfF0jwrFGtrIxr2ZH2dahwAgYZ1/xnhw0JqPH3H7O8YYX86pPcSLpJZkSC 7ZYRdNU91v023mUvYSUmbDQDav88yBMbkXkpm8+jsvMyZAB9v8KArH6JCJuZi7vcTbO0ZQt3O pZJHrmQuBWrHRYWMK5XTpzvZ4KqjYNvxmvQhWTXlMZ0XeFbhpKRC+FbKI6r3UdSaX6jydo9Yl SZSyuOcEdbT3YJkEI+qul+IifLGIl1KE/cJ+ZeP3toQo1Rzmtk= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: From: Markus Elfring Date: Sat, 13 Jan 2018 14:40:44 +0100 The script "checkpatch.pl" pointed information out like the following. ERROR: do not use assignment in if condition Thus fix the affected source code places. Signed-off-by: Markus Elfring --- fs/nfsd/export.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c index 3b035a87e4c4..8ab1383bf3ea 100644 --- a/fs/nfsd/export.c +++ b/fs/nfsd/export.c @@ -90,7 +90,8 @@ static int expkey_parse(struct cache_detail *cd, char *mesg, int mlen) goto out; err = -EINVAL; - if ((len=qword_get(&mesg, buf, PAGE_SIZE)) <= 0) + len = qword_get(&mesg, buf, PAGE_SIZE); + if (len <= 0) goto out; err = -ENOENT; @@ -100,7 +101,8 @@ static int expkey_parse(struct cache_detail *cd, char *mesg, int mlen) dprintk("found domain %s\n", buf); err = -EINVAL; - if ((len=qword_get(&mesg, buf, PAGE_SIZE)) <= 0) + len = qword_get(&mesg, buf, PAGE_SIZE); + if (len <= 0) goto out; fsidtype = simple_strtoul(buf, &ep, 10); if (*ep) @@ -108,7 +110,9 @@ static int expkey_parse(struct cache_detail *cd, char *mesg, int mlen) dprintk("found fsidtype %d\n", fsidtype); if (key_len(fsidtype)==0) /* invalid type */ goto out; - if ((len=qword_get(&mesg, buf, PAGE_SIZE)) <= 0) + + len = qword_get(&mesg, buf, PAGE_SIZE); + if (len <= 0) goto out; dprintk("found fsid length %d\n", len); if (len != key_len(fsidtype)) @@ -538,7 +542,8 @@ static int svc_export_parse(struct cache_detail *cd, char *mesg, int mlen) /* path */ err = -EINVAL; - if ((len = qword_get(&mesg, buf, PAGE_SIZE)) <= 0) + len = qword_get(&mesg, buf, PAGE_SIZE); + if (len <= 0) goto out1; err = kern_path(buf, 0, &exp.ex_path); -- 2.15.1