In downgrade_table_extra, the return value is needed. When it
return failed, we should exit immediately.

Fixes: 7773df19c35f ("bcachefs: metadata version bucket_stripe_sectors")
Signed-off-by: Hongbo Li <[email protected]>
---
 fs/bcachefs/sb-downgrade.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/fs/bcachefs/sb-downgrade.c b/fs/bcachefs/sb-downgrade.c
index c7e4cdd3f6a5..6f0493f79959 100644
--- a/fs/bcachefs/sb-downgrade.c
+++ b/fs/bcachefs/sb-downgrade.c
@@ -353,7 +353,9 @@ int bch2_sb_downgrade_update(struct bch_fs *c)
                for (unsigned i = 0; i < src->nr_errors; i++)
                        dst->errors[i] = cpu_to_le16(src->errors[i]);
 
-               downgrade_table_extra(c, &table);
+               ret = downgrade_table_extra(c, &table);
+               if (ret)
+                       goto out;
 
                if (!dst->recovery_passes[0] &&
                    !dst->recovery_passes[1] &&
-- 
2.34.1


Reply via email to