From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793211883; b=OVfx5lVR3PNLLS4FfPrAqcRvdsQprWFpy37Y9Dr/uLvIaDUQOMAJUfJ/6wajXKNDtzbJ OwnWZojYUIK6JtLx0tGFZ2XP910ntAt4btD6ifd0jnRoQ0qTYYOInJ3q44fEkTZteF0AE RJLkMw9aDHqscM7eScr/yG5Zdi4WCAVbFCFlWWdLeug9DwCQohSB6pjahlF+YjWYmcnjT C9XAZD1BfAEQhWYXFLb8zSxTjfY7Vf/7bswtEqTWLo2SvboB2e6IwTX2hmNlO2z8eShch /dwwq24cNUuWYC98gHcpEvycTYyAcatGOYExGCIs0t8vL+F2gfMSDF2BdH4zMDQ3AKw== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793211883; h=Received:DKIM-Signature:From:To:Cc:Subject:Date:Message-ID:X-Mailer: In-Reply-To:References:User-Agent:X-stable:X-Patchwork-Hint: MIME-Version:Content-Transfer-Encoding; bh=CZGkbfPy5Z6myzyCw5DibGmzSU3QJFx4jPfLH/IFl0M=; b=yJP0byGhUGffPVA2OlviLrW6jF3FUqrAWo7UuZqL0EjpAwKnABLAGDcoTVDn4TBXwZ/j koZwo8ZUbveFXSfS52h++wvySX20thdMUHO+IP+KvJtnnYPR/pBOjTixIb4XYSZyAvAWC cZm10AyX3soA/8qIIxKS0cX3SF7jSPvVugT6I/gi/OdWmZqDWT5Sa4ZQph+/wb9Azu8FJ tK6LnENjdgqfD8NGkO5u2xLFnWX+SLAVpWCAjzjEZfeXrv2WmvgH5ci7J2pIL+ph+gY+B b4ibbTHQoLgN+ASODggyEtvPpClkTGEIjA2N3NVgLGt/MLAxECaokvtoFptTil5H8Ag== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=linuxfoundation.org header.i=@linuxfoundation.org header.a=rsa-sha256 header.s=korg header.b=Akyocd4n; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9F1D7606A31D; Wed, 28 Oct 2026 18:24:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key, unprotected) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.a=rsa-sha256 header.s=korg header.b=Akyocd4n Received: by smtp.kernel.org (Postfix) with ESMTPSA id F092B1F01828; Wed, 28 Oct 2026 18:24:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1793211883; bh=CZGkbfPy5Z6myzyCw5DibGmzSU3QJFx4jPfLH/IFl0M=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=Akyocd4noguL7SlDDGVJUTmlqf6Vvm4FwwdjKet+OWOw5HUrJJytHdsQMd4eigvbG 0nmn0bcYh0F4wctPbuhzkIP80bS2/ywWg7wjUUKBoeqp6mDv/Hk7xYbvWo/5x66DEh mmnZD1+eUEQtX3YYgv4Jp7NQcWYAEZ7gZ1cwLiDY= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, "Christian Brauner (Amutable)" , Sasha Levin Subject: [PATCH 5.15 534/571] unicode: free the map after a grace period Date: Wed, 28 Oct 2026 18:01:21 +0100 Message-ID: <20261028170020.702894827@linuxfoundation.org> X-Mailer: git-send-email 2.56.0 In-Reply-To: <20261028170008.009143722@linuxfoundation.org> References: <20261028170008.009143722@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore X-Mailing-List: patches@lists.linux.dev List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 5.15-stable review patch. If anyone has any objections, please let me know. ------------------ From: Christian Brauner [ Upstream commit a96c5299bd7305eb984377681eb3222a57376405 ] generic_ci_d_hash() and generic_ci_d_compare() read sb->s_encoding and dereference the unicode map while a pathwalk in rcu mode holds nothing but rcu_read_lock(). The filesystems that use them, ext4, f2fs and tmpfs, drop the map with utf8_unload() from ->put_super(), which runs when the last reference to the mount goes away. A lookup that is still inside such a filesystem at that point reads a freed map. With a pathwalk held up in utf8_casefold_hash() while the last reference to a casefolded tmpfs goes away: BUG: KASAN: slab-use-after-free in utf8nlookup+0x74c/0x950 Read of size 8 at addr ff110001030fe658 by task walker/98 Call Trace: utf8nlookup+0x74c/0x950 utf8byte+0x160/0x1560 utf8_casefold_hash+0x167/0x230 generic_ci_d_hash+0xdd/0x1c0 link_path_walk+0x11ad/0x1d30 path_lookupat+0x63/0xb00 filename_lookup+0x1f1/0x5b0 vfs_statx+0xd7/0x3b0 [...] Allocated by task 94: __kmalloc_cache_noprof+0x249/0x560 utf8_load+0x4e/0x570 shmem_parse_opt_casefold.isra.0+0x92/0x270 [...] Freed by task 97: kfree+0x1db/0x5b0 shmem_put_super+0x72/0x170 generic_shutdown_super+0x14c/0x530 kill_anon_super+0x3a/0x60 deactivate_locked_super+0x9c/0xf0 cleanup_mnt+0x9f/0x130 So far that has been hard to hit. umount(2) waits for a grace period in namespace_unlock() before it drops the mount, so only a mount whose last reference goes away elsewhere, like the close of a file or a chdir out of a lazily unmounted directory, can race a walker. So it is reachable. Delay the free until a grace period has passed. The map and the data table it holds a reference to then outlive any pathwalk that found the superblock before the mount went away. The reference on the data table module is dropped from the rcu callback as well. utf8-core is built in whenever CONFIG_UNICODE is set, so there is no module exit that would have to wait for the callbacks. Overlayfs points the s_encoding of an overlay at the map of its first layer without taking a reference of its own. That stays correct. The layer is held by the overlay, so its map is dropped only after the overlay is gone. Fixes: b886ee3e778e ("ext4: Support case-insensitive file name lookups") Cc: stable@vger.kernel.org Link: https://patch.msgid.link/20261009-work-fixes-rcu-v1-6-7446d2d335e6@kernel.org Signed-off-by: Christian Brauner (Amutable) [ sashal: Replaced the custom RCU callback with kfree_rcu() because built-in Unicode tables need no module reference release. ] Signed-off-by: Sasha Levin --- fs/unicode/utf8-core.c | 9 ++++++++- include/linux/unicode.h | 1 + 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/fs/unicode/utf8-core.c b/fs/unicode/utf8-core.c index dc25823bfed96..d946ccfd90181 100644 --- a/fs/unicode/utf8-core.c +++ b/fs/unicode/utf8-core.c @@ -3,6 +3,7 @@ #include #include #include +#include #include #include #include @@ -227,9 +228,15 @@ struct unicode_map *utf8_load(const char *version) } EXPORT_SYMBOL(utf8_load); +/* + * A pathwalk in rcu mode may still be in ->d_hash() or ->d_compare() of a + * filesystem that is being torn down and read the map through its + * superblock, so the map has to outlive the grace period. + */ void utf8_unload(struct unicode_map *um) { - kfree(um); + if (um) + kfree_rcu(um, rcu); } EXPORT_SYMBOL(utf8_unload); diff --git a/include/linux/unicode.h b/include/linux/unicode.h index 74484d44c7554..b82625a023688 100644 --- a/include/linux/unicode.h +++ b/include/linux/unicode.h @@ -8,6 +8,7 @@ struct unicode_map { const char *charset; int version; + struct rcu_head rcu; }; int utf8_validate(const struct unicode_map *um, const struct qstr *str); -- 2.53.0