bcachefs: Fix reference to invalid bucket in copygc

Use bch2_dev_bucket_tryget() instead of bch2_dev_tryget() before
checking the bucket bitmap.

Reported-by: syzbot+3168625f36f4a539237e@syzkaller.appspotmail.com
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>

diff --git a/fs/bcachefs/movinggc.c b/fs/bcachefs/movinggc.c
index 27e68d470ad0959928bb59fbfc610010353aa09c..5e6de91a87630d324f3d799925c8cac36293e93a 100644 (file)
--- a/fs/bcachefs/movinggc.c
+++ b/fs/bcachefs/movinggc.c
@@ -71,7 +71,7 @@ static int bch2_bucket_is_movable(struct btree_trans *trans,
        if (ret)
                return ret;
 
-       struct bch_dev *ca = bch2_dev_tryget(c, k.k->p.inode);
+       struct bch_dev *ca = bch2_dev_bucket_tryget(c, k.k->p);
        if (!ca)
                goto out;