xfs_db: don't abort when bmapping on a non-extents/bmbt fork

We're going to introduce new fork formats, so let's fix the problem that
xfs_db's bmap command aborts when the fork format isn't one of the
existing ones.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>

diff --git a/db/bmap.c b/db/bmap.c
index d63aa9ef9c015b2e42a5440c362cbf8082234561..76acb410f4baa1bd33d39859c4512e38dee159f7 100644 (file)
--- a/db/bmap.c
+++ b/db/bmap.c
@@ -65,16 +65,18 @@ bmap(
        fmt = (enum xfs_dinode_fmt)XFS_DFORK_FORMAT(dip, whichfork);
        typ = whichfork == XFS_DATA_FORK ? TYP_BMAPBTD : TYP_BMAPBTA;
        ASSERT(typtab[typ].typnm == typ);
-       ASSERT(fmt == XFS_DINODE_FMT_LOCAL || fmt == XFS_DINODE_FMT_EXTENTS ||
-               fmt == XFS_DINODE_FMT_BTREE);
-       if (fmt == XFS_DINODE_FMT_EXTENTS) {
+       switch (fmt) {
+       case XFS_DINODE_FMT_LOCAL:
+               break;
+       case XFS_DINODE_FMT_EXTENTS:
                nextents = xfs_dfork_nextents(dip, whichfork);
                xp = (xfs_bmbt_rec_t *)XFS_DFORK_PTR(dip, whichfork);
                for (ep = xp; ep < &xp[nextents] && n < nex; ep++) {
                        if (!bmap_one_extent(ep, &curoffset, eoffset, &n, bep))
                                break;
                }
-       } else if (fmt == XFS_DINODE_FMT_BTREE) {
+               break;
+       case XFS_DINODE_FMT_BTREE:
                push_cur();
                rblock = (xfs_bmdr_block_t *)XFS_DFORK_PTR(dip, whichfork);
                fsize = XFS_DFORK_SIZE(dip, mp, whichfork);
@@ -114,6 +116,13 @@ bmap(
                        block = (struct xfs_btree_block *)iocur_top->data;
                }
                pop_cur();
+               break;
+       default:
+               dbprintf(
+ _("%s fork format %u does not support indexable blocks\n"),
+                               whichfork == XFS_DATA_FORK ? "data" : "attr",
+                               fmt);
+               break;
        }
        pop_cur();
        *nexp = n;