]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
ext4: use ERR_CAST to return an error-valued pointer
authorYu Jiaoliang <yujiaoliang@vivo.com>
Fri, 20 Sep 2024 02:14:40 +0000 (10:14 +0800)
committerTheodore Ts'o <tytso@mit.edu>
Wed, 13 Nov 2024 04:54:14 +0000 (23:54 -0500)
Instead of directly casting and returning an error-valued pointer,
use ERR_CAST to make the error handling more explicit and improve
code clarity.

Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com>
Link: https://patch.msgid.link/20240920021440.1959243-1-yujiaoliang@vivo.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/namei.c

index 08d15cd2b5946fbadb92962891026067a9e269e3..1012781ae9b4770f4752985768aa1935b4c8b09f 100644 (file)
@@ -1747,7 +1747,7 @@ static struct buffer_head * ext4_dx_find_entry(struct inode *dir,
 #endif
        frame = dx_probe(fname, dir, NULL, frames);
        if (IS_ERR(frame))
-               return (struct buffer_head *) frame;
+               return ERR_CAST(frame);
        do {
                block = dx_get_block(frame->at);
                bh = ext4_read_dirblock(dir, block, DIRENT_HTREE);
@@ -1952,7 +1952,7 @@ static struct ext4_dir_entry_2 *do_split(handle_t *handle, struct inode *dir,
        if (IS_ERR(bh2)) {
                brelse(*bh);
                *bh = NULL;
-               return (struct ext4_dir_entry_2 *) bh2;
+               return ERR_CAST(bh2);
        }
 
        BUFFER_TRACE(*bh, "get_write_access");