btrfs: fix ordered extent split error handling in btrfs_dio_submit_io
authorChristoph Hellwig <hch@lst.de>
Fri, 14 Jul 2023 08:42:41 +0000 (10:42 +0200)
committerDavid Sterba <dsterba@suse.com>
Tue, 18 Jul 2023 01:14:01 +0000 (03:14 +0200)
commit7cad645ebf20d777b2a48750ebd80fd81593b78c
tree1c720cd300c25c5580aa02b43960470668bf096c
parent17b17fcd6d446b95904a6929c40012ee7f0afc0c
btrfs: fix ordered extent split error handling in btrfs_dio_submit_io

When the call to btrfs_extract_ordered_extent in btrfs_dio_submit_io
fails to allocate memory for a new ordered_extent, it calls into the
btrfs_dio_end_io for error handling.  btrfs_dio_end_io then assumes that
bbio->ordered is set because it is supposed to be at this point, except
for this error handling corner case.  Try to not overload the
btrfs_dio_end_io with error handling of a bio in a non-canonical state,
and instead call btrfs_finish_ordered_extent and iomap_dio_bio_end_io
directly for this error case.

Reported-by: syzbot <syzbot+5b82f0e951f8c2bcdb8f@syzkaller.appspotmail.com>
Fixes: b41b6f6937dc ("btrfs: use btrfs_finish_ordered_extent to complete direct writes")
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Tested-by: syzbot <syzbot+5b82f0e951f8c2bcdb8f@syzkaller.appspotmail.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/inode.c