Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753519AbdHUBqa convert rfc822-to-8bit (ORCPT ); Sun, 20 Aug 2017 21:46:30 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42180 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753447AbdHUBq2 (ORCPT ); Sun, 20 Aug 2017 21:46:28 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 99BBB356CE Authentication-Results: ext-mx06.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx06.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=zyan@redhat.com Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\)) Subject: Re: [PATCH 0/3] Ceph: Adjustments for some function implementations From: "Yan, Zheng" In-Reply-To: Date: Mon, 21 Aug 2017 09:46:54 +0800 Cc: ceph-devel@vger.kernel.org, Ilya Dryomov , Sage Weil , LKML , kernel-janitors@vger.kernel.org Content-Transfer-Encoding: 8BIT Message-Id: <621A6044-0445-42A4-AE39-E3E300833A7F@redhat.com> References: To: SF Markus Elfring X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Mon, 21 Aug 2017 01:46:28 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 980 Lines: 34 > On 21 Aug 2017, at 02:37, SF Markus Elfring wrote: > > From: Markus Elfring > Date: Sun, 20 Aug 2017 20:32:10 +0200 > > Three update suggestions were taken into account > from static source code analysis. > > Markus Elfring (3): > Delete an error message for a failed memory allocation in __get_or_create_frag() > Delete an unnecessary return statement in update_dentry_lease() > Adjust 36 checks for null pointers > > fs/ceph/addr.c | 2 +- > fs/ceph/cache.c | 2 +- > fs/ceph/caps.c | 4 ++-- > fs/ceph/debugfs.c | 2 +- > fs/ceph/file.c | 2 +- > fs/ceph/inode.c | 14 +++++--------- > fs/ceph/mds_client.c | 22 +++++++++++----------- > fs/ceph/mdsmap.c | 6 +++--- > fs/ceph/super.c | 18 +++++++++--------- > fs/ceph/xattr.c | 8 ++++---- > 10 files changed, 38 insertions(+), 42 deletions(-) > > — Whole series applied, thanks Yan, Zheng > 2.14.0 >