bcachefs: Fix __btree_trans_peek_key_cache()
authorKent Overstreet <kent.overstreet@linux.dev>
Tue, 20 Dec 2022 15:51:20 +0000 (10:51 -0500)
committerKent Overstreet <kent.overstreet@linux.dev>
Sun, 22 Oct 2023 21:09:47 +0000 (17:09 -0400)
We were returning a pointer to a variable on the stack - oops.

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
fs/bcachefs/btree_iter.c

index 8a18b55cab261cc74daf1c1f632d2fc5412b5aca..c95dbeaaceab4db91e9e0c56510dd899ba7ba4a9 100644 (file)
@@ -1851,6 +1851,7 @@ struct bkey_s_c __btree_trans_peek_key_cache(struct btree_iter *iter, struct bpo
        struct btree_trans *trans = iter->trans;
        struct bch_fs *c = trans->c;
        struct bkey u;
+       struct bkey_s_c k;
        int ret;
 
        if (!bch2_btree_key_cache_find(c, iter->btree_id, pos))
@@ -1870,7 +1871,12 @@ struct bkey_s_c __btree_trans_peek_key_cache(struct btree_iter *iter, struct bpo
 
        btree_path_set_should_be_locked(iter->key_cache_path);
 
-       return bch2_btree_path_peek_slot(iter->key_cache_path, &u);
+       k = bch2_btree_path_peek_slot(iter->key_cache_path, &u);
+       if (k.k && !bkey_err(k)) {
+               iter->k = u;
+               k.k = &iter->k;
+       }
+       return k;
 }
 
 static noinline