diff options
author | Jeongjun Park <aha310510@gmail.com> | 2024-10-24 01:13:45 +0900 |
---|---|---|
committer | Kent Overstreet <kent.overstreet@linux.dev> | 2024-10-25 13:17:06 -0400 |
commit | a25a83de45b435cf89e55c7fb8733f83c7826004 (patch) | |
tree | 2a7ba3448fc5df63cafef80f3bf87664fb63bb9d /fs | |
parent | 5c41f75d1b921b9eaf79588cdd3b22b00fb4ec52 (diff) |
bcachefs: fix null-ptr-deref in have_stripes()
c->btree_roots_known[i].b can be NULL. In this case, a NULL pointer dereference
occurs, so you need to add code to check the variable.
Reported-by: syzbot+b468b9fef56949c3b528@syzkaller.appspotmail.com
Fixes: 7773df19c35f ("bcachefs: metadata version bucket_stripe_sectors")
Signed-off-by: Jeongjun Park <aha310510@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/bcachefs/sb-downgrade.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/bcachefs/sb-downgrade.c b/fs/bcachefs/sb-downgrade.c index ae715ff658e8..8767c33c2b51 100644 --- a/fs/bcachefs/sb-downgrade.c +++ b/fs/bcachefs/sb-downgrade.c @@ -143,6 +143,9 @@ UPGRADE_TABLE() static int have_stripes(struct bch_fs *c) { + if (IS_ERR_OR_NULL(c->btree_roots_known[BTREE_ID_stripes].b)) + return 0; + return !btree_node_fake(c->btree_roots_known[BTREE_ID_stripes].b); } |