xfs: validate explicit directory free block owners

Source kernel commit: fe6c9f8e48e0dcbfc3dba17edd88490c8579b34b

Port the existing directory freespace block header checking function to
accept an owner number instead of an xfs_inode, then update the
callsites to use xfs_da_args.owner when possible.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>

diff --git a/libxfs/xfs_dir2_leaf.c b/libxfs/xfs_dir2_leaf.c
index 1c12b5a6644dfea0adf989b6bf91606f9634b362..7c0bba512881423a69240db9123906f6aa15c552 100644 (file)
--- a/libxfs/xfs_dir2_leaf.c
+++ b/libxfs/xfs_dir2_leaf.c
@@ -1804,7 +1804,8 @@ xfs_dir2_node_to_leaf(
        /*
         * Read the freespace block.
         */
-       error = xfs_dir2_free_read(tp, dp,  args->geo->freeblk, &fbp);
+       error = xfs_dir2_free_read(tp, dp, args->owner, args->geo->freeblk,
+                       &fbp);
        if (error)
                return error;
        xfs_dir2_free_hdr_from_disk(mp, &freehdr, fbp->b_addr);

diff --git a/libxfs/xfs_dir2_node.c b/libxfs/xfs_dir2_node.c
index 690407374187ff0ddc7b9ad29e96f063ee90f6d5..c94d00eb99c8743859ea37d0088b3cfbc87cb159 100644 (file)
--- a/libxfs/xfs_dir2_node.c
+++ b/libxfs/xfs_dir2_node.c
@@ -172,11 +172,11 @@ const struct xfs_buf_ops xfs_dir3_free_buf_ops = {
 /* Everything ok in the free block header? */
 static xfs_failaddr_t
 xfs_dir3_free_header_check(
-       struct xfs_inode        *dp,
-       xfs_dablk_t             fbno,
-       struct xfs_buf          *bp)
+       struct xfs_buf          *bp,
+       xfs_ino_t               owner,
+       xfs_dablk_t             fbno)
 {
-       struct xfs_mount        *mp = dp->i_mount;
+       struct xfs_mount        *mp = bp->b_mount;
        int                     maxbests = mp->m_dir_geo->free_max_bests;
        unsigned int            firstdb;
 
@@ -192,7 +192,7 @@ xfs_dir3_free_header_check(
                        return __this_address;
                if (be32_to_cpu(hdr3->nvalid) < be32_to_cpu(hdr3->nused))
                        return __this_address;
-               if (be64_to_cpu(hdr3->hdr.owner) != dp->i_ino)
+               if (be64_to_cpu(hdr3->hdr.owner) != owner)
                        return __this_address;
        } else {
                struct xfs_dir2_free_hdr *hdr = bp->b_addr;
@@ -211,6 +211,7 @@ static int
 __xfs_dir3_free_read(
        struct xfs_trans        *tp,
        struct xfs_inode        *dp,
+       xfs_ino_t               owner,
        xfs_dablk_t             fbno,
        unsigned int            flags,
        struct xfs_buf          **bpp)
@@ -224,7 +225,7 @@ __xfs_dir3_free_read(
                return err;
 
        /* Check things that we can't do in the verifier. */
-       fa = xfs_dir3_free_header_check(dp, fbno, *bpp);
+       fa = xfs_dir3_free_header_check(*bpp, owner, fbno);
        if (fa) {
                __xfs_buf_mark_corrupt(*bpp, fa);
                xfs_trans_brelse(tp, *bpp);
@@ -296,20 +297,23 @@ int
 xfs_dir2_free_read(
        struct xfs_trans        *tp,
        struct xfs_inode        *dp,
+       xfs_ino_t               owner,
        xfs_dablk_t             fbno,
        struct xfs_buf          **bpp)
 {
-       return __xfs_dir3_free_read(tp, dp, fbno, 0, bpp);
+       return __xfs_dir3_free_read(tp, dp, owner, fbno, 0, bpp);
 }
 
 static int
 xfs_dir2_free_try_read(
        struct xfs_trans        *tp,
        struct xfs_inode        *dp,
+       xfs_ino_t               owner,
        xfs_dablk_t             fbno,
        struct xfs_buf          **bpp)
 {
-       return __xfs_dir3_free_read(tp, dp, fbno, XFS_DABUF_MAP_HOLE_OK, bpp);
+       return __xfs_dir3_free_read(tp, dp, owner, fbno, XFS_DABUF_MAP_HOLE_OK,
+                       bpp);
 }
 
 static int
@@ -714,7 +718,7 @@ xfs_dir2_leafn_lookup_for_addname(
                                if (curbp)
                                        xfs_trans_brelse(tp, curbp);
 
-                               error = xfs_dir2_free_read(tp, dp,
+                               error = xfs_dir2_free_read(tp, dp, args->owner,
                                                xfs_dir2_db_to_da(args->geo,
                                                                  newfdb),
                                                &curbp);
@@ -1353,8 +1357,8 @@ xfs_dir2_leafn_remove(
                 * read in the free block.
                 */
                fdb = xfs_dir2_db_to_fdb(geo, db);
-               error = xfs_dir2_free_read(tp, dp, xfs_dir2_db_to_da(geo, fdb),
-                                          &fbp);
+               error = xfs_dir2_free_read(tp, dp, args->owner,
+                               xfs_dir2_db_to_da(geo, fdb), &fbp);
                if (error)
                        return error;
                free = fbp->b_addr;
@@ -1713,7 +1717,7 @@ xfs_dir2_node_add_datablk(
         * that was just allocated.
         */
        fbno = xfs_dir2_db_to_fdb(args->geo, *dbno);
-       error = xfs_dir2_free_try_read(tp, dp,
+       error = xfs_dir2_free_try_read(tp, dp, args->owner,
                               xfs_dir2_db_to_da(args->geo, fbno), &fbp);
        if (error)
                return error;
@@ -1860,7 +1864,7 @@ xfs_dir2_node_find_freeblk(
                 * so this might not succeed.  This should be really rare, so
                 * there's no reason to avoid it.
                 */
-               error = xfs_dir2_free_try_read(tp, dp,
+               error = xfs_dir2_free_try_read(tp, dp, args->owner,
                                xfs_dir2_db_to_da(args->geo, fbno),
                                &fbp);
                if (error)
@@ -2299,7 +2303,7 @@ xfs_dir2_node_trim_free(
        /*
         * Read the freespace block.
         */
-       error = xfs_dir2_free_try_read(tp, dp, fo, &bp);
+       error = xfs_dir2_free_try_read(tp, dp, args->owner, fo, &bp);
        if (error)
                return error;
        /*

diff --git a/libxfs/xfs_dir2_priv.h b/libxfs/xfs_dir2_priv.h
index adbc544c9befab3167b9ed7933edf21ac4b861a7..3befb32509fa4466a32eceebc066b6fef0870951 100644 (file)
--- a/libxfs/xfs_dir2_priv.h
+++ b/libxfs/xfs_dir2_priv.h
@@ -155,8 +155,8 @@ extern int xfs_dir2_node_removename(struct xfs_da_args *args);
 extern int xfs_dir2_node_replace(struct xfs_da_args *args);
 extern int xfs_dir2_node_trim_free(struct xfs_da_args *args, xfs_fileoff_t fo,
                int *rvalp);
-extern int xfs_dir2_free_read(struct xfs_trans *tp, struct xfs_inode *dp,
-               xfs_dablk_t fbno, struct xfs_buf **bpp);
+int xfs_dir2_free_read(struct xfs_trans *tp, struct xfs_inode *dp,
+               xfs_ino_t owner, xfs_dablk_t fbno, struct xfs_buf **bpp);
 
 /* xfs_dir2_sf.c */
 xfs_ino_t xfs_dir2_sf_get_ino(struct xfs_mount *mp, struct xfs_dir2_sf_hdr *hdr,