From patchwork Thu Dec 15 08:18:19 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: ChenXiaoSong X-Patchwork-Id: 33536 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:e747:0:0:0:0:0 with SMTP id c7csp196210wrn; Wed, 14 Dec 2022 23:26:43 -0800 (PST) X-Google-Smtp-Source: AA0mqf6MdUXm6d3XXraC226dE8VtW5fMuNgZkbZ6sbUmlrYD+JUIi4UZ5qE1gOJp09IdvEJ+d9P8 X-Received: by 2002:a17:906:b14b:b0:78d:f459:719e with SMTP id bt11-20020a170906b14b00b0078df459719emr8970392ejb.73.1671089203027; Wed, 14 Dec 2022 23:26:43 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1671089203; cv=none; d=google.com; s=arc-20160816; b=Wuxgr8tQrwyf5aes/KDCDUMXg/bOO40GjuDObfl0FmmXY3XZT0yZ0jI9LoNMrolWno x5ilY+4PqXRbskJgeRrhJaqiTOcWuuzMVFMtLJZyT+jB5Un6QHHYLMa/mKy4MZHj242P XXOK7bqAMuWtAcIVItEW21MalfTrfxRTiueVj8fcGnAu0RWtFZDwIUZzyhKcLhtdN/cE DrkmZaAABLXew92iRsnvdCmHaHuFFaaupwkZelM9/YCarX7oKRHUqMKivmQZ2VmnjfM5 ICX02iq/Zfpt2Rjlz2FdcUBBEPjRVEp9wvwdx/LnbFzkD8R1P0JcNh/Ln58GLXlugjQ6 ZNmg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=bvNRxhfdK1TBxOwDt5ixQiqAKmSL1k5lFLpg5h0ZvWk=; b=D0VvDSHWKhQNNxuQK5TxZaY5h4SUY/l3xlcW/3IaUaksks52n7cbMk6sorZaz68IKK +qks12XRhlJNk9Ow9P2K0yqR6mdxYh4600KCpujdxXjpEQhr5pxdZfN/HoDiqbCtdmF0 9EwwZFN1LAkrePZETO0CLO6EbdnjB3JCfaNPSnhcStG8mZmoH5cl5rxzIBs3jKaQ4u9l W5ocVxfI3KVwQ7tyvUQu3XzuLHXSLswo+kFKUc6xaeqgowuNMDyL40kSw/OJqvgUWA7e i4nPgaoMWrZnKgFu9jl3WFQ76iqLBmw6QPKEvvhL2HR6Y7uzqy1Ymj+pfQrxkeeZ6zQS IJ0g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n17-20020a170906089100b007bded759a0bsi10422950eje.892.2022.12.14.23.26.20; Wed, 14 Dec 2022 23:26:43 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230096AbiLOHPV (ORCPT + 99 others); Thu, 15 Dec 2022 02:15:21 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36570 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229600AbiLOHOo (ORCPT ); Thu, 15 Dec 2022 02:14:44 -0500 Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 969033134E; Wed, 14 Dec 2022 23:14:37 -0800 (PST) Received: from kwepemm600015.china.huawei.com (unknown [172.30.72.53]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4NXjzq0bNbzgZ2b; Thu, 15 Dec 2022 15:10:15 +0800 (CST) Received: from huawei.com (10.175.101.6) by kwepemm600015.china.huawei.com (7.193.23.52) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Thu, 15 Dec 2022 15:14:32 +0800 From: ChenXiaoSong To: , , , , , , , , , , , CC: , Subject: [PATCH v2 1/2] hfsplus: introduce hfsplus_init_inode() helper Date: Thu, 15 Dec 2022 16:18:19 +0800 Message-ID: <20221215081820.948990-2-chenxiaosong2@huawei.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20221215081820.948990-1-chenxiaosong2@huawei.com> References: <20221215081820.948990-1-chenxiaosong2@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.175.101.6] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To kwepemm600015.china.huawei.com (7.193.23.52) X-CFilter-Loop: Reflected X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1752264032179438235?= X-GMAIL-MSGID: =?utf-8?q?1752264032179438235?= Factor out hfsplus_init_inode(), No functional change. Signed-off-by: ChenXiaoSong --- fs/hfsplus/hfsplus_fs.h | 1 + fs/hfsplus/inode.c | 35 ++++++++++++++++++++--------------- 2 files changed, 21 insertions(+), 15 deletions(-) diff --git a/fs/hfsplus/hfsplus_fs.h b/fs/hfsplus/hfsplus_fs.h index 6aa919e59483..2aa719e00ae5 100644 --- a/fs/hfsplus/hfsplus_fs.h +++ b/fs/hfsplus/hfsplus_fs.h @@ -472,6 +472,7 @@ extern const struct dentry_operations hfsplus_dentry_operations; int hfsplus_write_begin(struct file *file, struct address_space *mapping, loff_t pos, unsigned len, struct page **pagep, void **fsdata); +void hfsplus_init_inode(struct hfsplus_inode_info *hip); struct inode *hfsplus_new_inode(struct super_block *sb, struct inode *dir, umode_t mode); void hfsplus_delete_inode(struct inode *inode); diff --git a/fs/hfsplus/inode.c b/fs/hfsplus/inode.c index 840577a0c1e7..d921b32d292e 100644 --- a/fs/hfsplus/inode.c +++ b/fs/hfsplus/inode.c @@ -379,22 +379,8 @@ static const struct file_operations hfsplus_file_operations = { .unlocked_ioctl = hfsplus_ioctl, }; -struct inode *hfsplus_new_inode(struct super_block *sb, struct inode *dir, - umode_t mode) +void hfsplus_init_inode(struct hfsplus_inode_info *hip) { - struct hfsplus_sb_info *sbi = HFSPLUS_SB(sb); - struct inode *inode = new_inode(sb); - struct hfsplus_inode_info *hip; - - if (!inode) - return NULL; - - inode->i_ino = sbi->next_cnid++; - inode_init_owner(&init_user_ns, inode, dir, mode); - set_nlink(inode, 1); - inode->i_mtime = inode->i_atime = inode->i_ctime = current_time(inode); - - hip = HFSPLUS_I(inode); INIT_LIST_HEAD(&hip->open_dir_list); spin_lock_init(&hip->open_dir_lock); mutex_init(&hip->extents_lock); @@ -412,6 +398,25 @@ struct inode *hfsplus_new_inode(struct super_block *sb, struct inode *dir, hip->phys_size = 0; hip->fs_blocks = 0; hip->rsrc_inode = NULL; +} + +struct inode *hfsplus_new_inode(struct super_block *sb, struct inode *dir, + umode_t mode) +{ + struct hfsplus_sb_info *sbi = HFSPLUS_SB(sb); + struct inode *inode = new_inode(sb); + struct hfsplus_inode_info *hip; + + if (!inode) + return NULL; + + inode->i_ino = sbi->next_cnid++; + inode_init_owner(&init_user_ns, inode, dir, mode); + set_nlink(inode, 1); + inode->i_mtime = inode->i_atime = inode->i_ctime = current_time(inode); + + hip = HFSPLUS_I(inode); + hfsplus_init_inode(hip); if (S_ISDIR(inode->i_mode)) { inode->i_size = 2; sbi->folder_count++; From patchwork Thu Dec 15 08:18:20 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: ChenXiaoSong X-Patchwork-Id: 33534 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:e747:0:0:0:0:0 with SMTP id c7csp196020wrn; Wed, 14 Dec 2022 23:26:09 -0800 (PST) X-Google-Smtp-Source: AA0mqf7Ok2t/OuDaDZlviiQAV1LzqWubE+PqtQzHlAcA4I5SIkauCf8fmF+QEHuIhSKHmqCNlr0W X-Received: by 2002:a17:906:dd9:b0:7c0:dfba:54d3 with SMTP id p25-20020a1709060dd900b007c0dfba54d3mr21838633eji.20.1671089169512; Wed, 14 Dec 2022 23:26:09 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1671089169; cv=none; d=google.com; s=arc-20160816; b=rphMU16HbyNsdI4Iz9Shd2xjLlbevFjMN0PTDvMbO0WO3oH53I+iGPLVZccmza72fl FRVdxb+5p+8HB10/V8tQDB4ma5zz842rv5WDaH6PODCbdo5yy1Kl1LWVxvHTgRA9GX+M ZzXSo6pPpyJ3iQNxGQcV5zbpT49fVIpD13OjDjNdUvtmS0/WWesYUOCX5DZ1ZQ9WMyUu VxJjb0f0Oi3Nx+pmG1HUwGz3nLSbl5oZeQcxy+Hcm2FA+rvyPqalhUqyR2ydpwPM+8tN kiNQnVwmT94/FaaFuy3olRobJnAeCLYvKcwxKg5pPGWezVIB9C/zX1+2ibv6DJj9r5xf nUsA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=+WInHopSng6iPVLBnnP0+00GOZ+isqb/DIuw/t1VID4=; b=UsHkshL7RGhssbqnZM4ocdtDrbOcsPWKAftQGWxxrAvWO2Dk6ZZWXMYFC1yx6GcrHU A4SCxpoK5jUTeVDAeqZjveoeAeGsch0fzL97PveKhpWKrYYUX4VCddm8tfvzmHtvAwzQ G4EVSWA7XQjV/2FyL3pz++ajBhyANP/Y6KrbK3njZrXu5Qk6dAmVlYr+Mz0a+Fy7zAmT FMlfEBE9s7CNexjjA+9VXP2WhV9y1h6xelChydtmAGHi4DFr5ulAUvlwsuEWY4UpWSXf Co+wR7pEOoalOWzbDryETRwiFhIlmANJfK2sImk2zGMqr3w3uIuGFcjtNAfAYKKZcXOC lunw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id l7-20020a170906794700b007ae1723e8d7si15100795ejo.610.2022.12.14.23.25.45; Wed, 14 Dec 2022 23:26:09 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230062AbiLOHPQ (ORCPT + 99 others); Thu, 15 Dec 2022 02:15:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36540 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230111AbiLOHOm (ORCPT ); Thu, 15 Dec 2022 02:14:42 -0500 Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6090329351; Wed, 14 Dec 2022 23:14:36 -0800 (PST) Received: from kwepemm600015.china.huawei.com (unknown [172.30.72.53]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4NXk3f2GrTzmWbP; Thu, 15 Dec 2022 15:13:34 +0800 (CST) Received: from huawei.com (10.175.101.6) by kwepemm600015.china.huawei.com (7.193.23.52) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Thu, 15 Dec 2022 15:14:33 +0800 From: ChenXiaoSong To: , , , , , , , , , , , CC: , Subject: [PATCH v2 2/2] hfsplus: fix uninit-value in hfsplus_delete_cat() Date: Thu, 15 Dec 2022 16:18:20 +0800 Message-ID: <20221215081820.948990-3-chenxiaosong2@huawei.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20221215081820.948990-1-chenxiaosong2@huawei.com> References: <20221215081820.948990-1-chenxiaosong2@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.175.101.6] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To kwepemm600015.china.huawei.com (7.193.23.52) X-CFilter-Loop: Reflected X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1752263997409193410?= X-GMAIL-MSGID: =?utf-8?q?1752263997409193410?= Syzkaller reported BUG as follows: ===================================================== BUG: KMSAN: uninit-value in hfsplus_subfolders_dec fs/hfsplus/catalog.c:248 [inline] BUG: KMSAN: uninit-value in hfsplus_delete_cat+0x1207/0x14d0 fs/hfsplus/catalog.c:419 hfsplus_subfolders_dec fs/hfsplus/catalog.c:248 [inline] hfsplus_delete_cat+0x1207/0x14d0 fs/hfsplus/catalog.c:419 hfsplus_rmdir+0x141/0x3d0 fs/hfsplus/dir.c:425 hfsplus_rename+0x102/0x2e0 fs/hfsplus/dir.c:545 vfs_rename+0x1e4c/0x2800 fs/namei.c:4779 do_renameat2+0x173d/0x1dc0 fs/namei.c:4930 __do_sys_renameat2 fs/namei.c:4963 [inline] __se_sys_renameat2 fs/namei.c:4960 [inline] __ia32_sys_renameat2+0x14b/0x1f0 fs/namei.c:4960 do_syscall_32_irqs_on arch/x86/entry/common.c:112 [inline] __do_fast_syscall_32+0xa2/0x100 arch/x86/entry/common.c:178 do_fast_syscall_32+0x33/0x70 arch/x86/entry/common.c:203 do_SYSENTER_32+0x1b/0x20 arch/x86/entry/common.c:246 entry_SYSENTER_compat_after_hwframe+0x70/0x82 Uninit was stored to memory at: hfsplus_subfolders_inc fs/hfsplus/catalog.c:232 [inline] hfsplus_create_cat+0x19e3/0x19f0 fs/hfsplus/catalog.c:314 hfsplus_mknod+0x1fd/0x560 fs/hfsplus/dir.c:494 hfsplus_mkdir+0x54/0x60 fs/hfsplus/dir.c:529 vfs_mkdir+0x62a/0x870 fs/namei.c:4036 do_mkdirat+0x466/0x7b0 fs/namei.c:4061 __do_sys_mkdirat fs/namei.c:4076 [inline] __se_sys_mkdirat fs/namei.c:4074 [inline] __ia32_sys_mkdirat+0xc4/0x120 fs/namei.c:4074 do_syscall_32_irqs_on arch/x86/entry/common.c:112 [inline] __do_fast_syscall_32+0xa2/0x100 arch/x86/entry/common.c:178 do_fast_syscall_32+0x33/0x70 arch/x86/entry/common.c:203 do_SYSENTER_32+0x1b/0x20 arch/x86/entry/common.c:246 entry_SYSENTER_compat_after_hwframe+0x70/0x82 Uninit was created at: __alloc_pages+0x9f1/0xe80 mm/page_alloc.c:5581 alloc_pages+0xaae/0xd80 mm/mempolicy.c:2285 alloc_slab_page mm/slub.c:1794 [inline] allocate_slab+0x1b5/0x1010 mm/slub.c:1939 new_slab mm/slub.c:1992 [inline] ___slab_alloc+0x10c3/0x2d60 mm/slub.c:3180 __slab_alloc mm/slub.c:3279 [inline] slab_alloc_node mm/slub.c:3364 [inline] slab_alloc mm/slub.c:3406 [inline] __kmem_cache_alloc_lru mm/slub.c:3413 [inline] kmem_cache_alloc_lru+0x6f3/0xb30 mm/slub.c:3429 alloc_inode_sb include/linux/fs.h:3125 [inline] hfsplus_alloc_inode+0x56/0xc0 fs/hfsplus/super.c:627 alloc_inode+0x83/0x440 fs/inode.c:259 iget_locked+0x2a1/0xe20 fs/inode.c:1286 hfsplus_iget+0x5f/0xb60 fs/hfsplus/super.c:64 hfsplus_btree_open+0x13b/0x1cf0 fs/hfsplus/btree.c:150 hfsplus_fill_super+0x12b0/0x2a80 fs/hfsplus/super.c:473 mount_bdev+0x508/0x840 fs/super.c:1401 hfsplus_mount+0x49/0x60 fs/hfsplus/super.c:641 legacy_get_tree+0x10c/0x280 fs/fs_context.c:610 vfs_get_tree+0xa1/0x500 fs/super.c:1531 do_new_mount+0x694/0x1580 fs/namespace.c:3040 path_mount+0x71a/0x1eb0 fs/namespace.c:3370 do_mount fs/namespace.c:3383 [inline] __do_sys_mount fs/namespace.c:3591 [inline] __se_sys_mount+0x734/0x840 fs/namespace.c:3568 __ia32_sys_mount+0xdf/0x140 fs/namespace.c:3568 do_syscall_32_irqs_on arch/x86/entry/common.c:112 [inline] __do_fast_syscall_32+0xa2/0x100 arch/x86/entry/common.c:178 do_fast_syscall_32+0x33/0x70 arch/x86/entry/common.c:203 do_SYSENTER_32+0x1b/0x20 arch/x86/entry/common.c:246 entry_SYSENTER_compat_after_hwframe+0x70/0x82 ===================================================== Fix this by initializing 'subfolders' of 'struct hfsplus_inode_info' in hfsplus_iget(). Link: https://syzkaller.appspot.com/bug?id=981f82f21b973f2f5663dfea581ff8cee1ddfef2 Suggested-by: Viacheslav Dubeyko Signed-off-by: ChenXiaoSong --- fs/hfsplus/super.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/fs/hfsplus/super.c b/fs/hfsplus/super.c index 122ed89ebf9f..5812818759dd 100644 --- a/fs/hfsplus/super.c +++ b/fs/hfsplus/super.c @@ -67,13 +67,7 @@ struct inode *hfsplus_iget(struct super_block *sb, unsigned long ino) if (!(inode->i_state & I_NEW)) return inode; - INIT_LIST_HEAD(&HFSPLUS_I(inode)->open_dir_list); - spin_lock_init(&HFSPLUS_I(inode)->open_dir_lock); - mutex_init(&HFSPLUS_I(inode)->extents_lock); - HFSPLUS_I(inode)->flags = 0; - HFSPLUS_I(inode)->extent_state = 0; - HFSPLUS_I(inode)->rsrc_inode = NULL; - atomic_set(&HFSPLUS_I(inode)->opencnt, 0); + hfsplus_init_inode(HFSPLUS_I(inode)); if (inode->i_ino >= HFSPLUS_FIRSTUSER_CNID || inode->i_ino == HFSPLUS_ROOT_CNID) {