From patchwork Tue Jul 24 13:01:55 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Miklos Szeredi X-Patchwork-Id: 10542137 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 0D34D14E2 for ; Tue, 24 Jul 2018 13:02:04 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id EF39D287AA for ; Tue, 24 Jul 2018 13:02:03 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id E364D287AC; Tue, 24 Jul 2018 13:02:03 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 919E1287AA for ; Tue, 24 Jul 2018 13:02:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388360AbeGXOIZ (ORCPT ); Tue, 24 Jul 2018 10:08:25 -0400 Received: from mail-wm0-f65.google.com ([74.125.82.65]:37177 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388356AbeGXOIZ (ORCPT ); Tue, 24 Jul 2018 10:08:25 -0400 Received: by mail-wm0-f65.google.com with SMTP id n11-v6so2465043wmc.2 for ; Tue, 24 Jul 2018 06:02:01 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=3BgljC+3PT0Ic4dJXbKfRELkvUnoZrlTurBgY1AEfa0=; b=hrmCgw6PFqaGDkPetcA+s/V2PtoIMAujvCEwxbbNge5uYTof1dKHrRxCcvwP9AdHIL cQsNsGOQQHGGHChU0Kkj4ivznB9yzak24EGsBM22yPR9PPICbBAkz22Jkd/hw4x6b8Im MmCuTjgtxqoNl+TRgAtCidRJBn7hUg+21Z7NEC97n2msooiA0vJu2AwwfIwDeCecSc+m nq+Ms5QtMCrp6smyqkQKEI/PM6qXRD9r+1qymeEGFRJ+dOg6zK25YkruvqiicKpeZtfR J1iPyPId7WYcFdwEnORV8lYt3Y/wm2mGtxqTlMBtoOS2wdbGWaJKCUqh3JBU1M1cwVuv JcCg== X-Gm-Message-State: AOUpUlEn2ijsMB+r59trNuXVZgIaAo1/5yQPeyD5GsxTRuTeAar5rfqC zs0313UHRXweZhJlBWLYF6p5WQ== X-Google-Smtp-Source: AAOMgpdZLDRjUaCiyl5k4X8a8ayBtWRoKnUAu6A6sH3GFZB1OgUiu2gKtWxK2A0o+VGmFsNKdaY5gA== X-Received: by 2002:a1c:7506:: with SMTP id o6-v6mr1945308wmc.60.1532437320892; Tue, 24 Jul 2018 06:02:00 -0700 (PDT) Received: from veci.piliscsaba.redhat.com (catv-212-96-48-140.catv.broadband.hu. [212.96.48.140]) by smtp.gmail.com with ESMTPSA id f20-v6sm1993358wmd.3.2018.07.24.06.01.59 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 24 Jul 2018 06:02:00 -0700 (PDT) From: Miklos Szeredi To: Al Viro Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: [PATCH v3] vfs: don't evict uninitialized inode Date: Tue, 24 Jul 2018 15:01:55 +0200 Message-Id: <20180724130155.20641-1-mszeredi@redhat.com> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180724064929.GD19722@shao2-debian> References: <20180724064929.GD19722@shao2-debian> Sender: linux-fsdevel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP iput() ends up calling ->evict() on new inode, which is not yet initialized by owning fs. So use destroy_inode() instead. Add to sb->s_inodes list only if inode is not in I_CREATING state (meaning that it wasn't allocated with new_inode(), which already does the insertion). Reported-by: Al Viro Signed-off-by: Miklos Szeredi Fixes: 80ea09a002bf ("vfs: factor out inode_insert5()") --- fs/inode.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/fs/inode.c b/fs/inode.c index 04dd7e0d5142..0aa5b29b6f87 100644 --- a/fs/inode.c +++ b/fs/inode.c @@ -1050,6 +1050,7 @@ struct inode *inode_insert5(struct inode *inode, unsigned long hashval, { struct hlist_head *head = inode_hashtable + hash(inode->i_sb, hashval); struct inode *old; + bool creating = inode->i_state & I_CREATING; again: spin_lock(&inode_hash_lock); @@ -1083,6 +1084,8 @@ struct inode *inode_insert5(struct inode *inode, unsigned long hashval, inode->i_state |= I_NEW; hlist_add_head(&inode->i_hash, head); spin_unlock(&inode->i_lock); + if (!creating) + inode_sb_list_add(inode); unlock: spin_unlock(&inode_hash_lock); @@ -1117,12 +1120,13 @@ struct inode *iget5_locked(struct super_block *sb, unsigned long hashval, struct inode *inode = ilookup5(sb, hashval, test, data); if (!inode) { - struct inode *new = new_inode(sb); + struct inode *new = alloc_inode(sb); if (new) { + new->i_state = 0; inode = inode_insert5(new, hashval, test, set, data); if (unlikely(inode != new)) - iput(new); + destroy_inode(new); } } return inode;