]> git.hungrycats.org Git - linux/commitdiff
ext4: no need to continue when the number of entries is 1
authorEdward Adam Davis <eadavis@qq.com>
Mon, 1 Jul 2024 14:25:03 +0000 (22:25 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 10 Oct 2024 10:00:50 +0000 (12:00 +0200)
commit 1a00a393d6a7fb1e745a41edd09019bd6a0ad64c upstream.

Fixes: ac27a0ec112a ("[PATCH] ext4: initial copy of files from ext3")
Reported-by: syzbot+ae688d469e36fb5138d0@syzkaller.appspotmail.com
Closes: https://syzkaller.appspot.com/bug?extid=ae688d469e36fb5138d0
Signed-off-by: Edward Adam Davis <eadavis@qq.com>
Reported-and-tested-by: syzbot+ae688d469e36fb5138d0@syzkaller.appspotmail.com
Link: https://patch.msgid.link/tencent_BE7AEE6C7C2D216CB8949CE8E6EE7ECC2C0A@qq.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Cc: stable@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/ext4/namei.c

index 12af7c7e09c1a76db7c729fbaa37e4eb1861b06f..b1942a1aff9edfdb6449ec3f601b3205ffd3fe58 100644 (file)
@@ -2046,7 +2046,7 @@ static struct ext4_dir_entry_2 *do_split(handle_t *handle, struct inode *dir,
                split = count/2;
 
        hash2 = map[split].hash;
-       continued = hash2 == map[split - 1].hash;
+       continued = split > 0 ? hash2 == map[split - 1].hash : 0;
        dxtrace(printk(KERN_INFO "Split block %lu at %x, %i/%i\n",
                        (unsigned long)dx_get_block(frame->at),
                                        hash2, split, count-split));