On Wed, Aug 01, 2018 at 07:15:24AM -0700, Darrick J. Wong wrote: > On Wed, Aug 01, 2018 at 02:12:02PM +0200, Carlos Maiolino wrote: > > On Tue, Jul 31, 2018 at 04:21:00PM -0700, Darrick J. Wong wrote: > > > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > > > > > Before we start processing what we /think/ is a da3 node block, actually > > > check the magic to make sure that we're looking at a node block. This > > > way we won't blow the asserts in _node_hdr_from_disk on corrupted > > > metadata. > > > > > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > > > The patch looks good, but why wouldn't merge it into the previous one? > > > > Both patches look reasonable to fit into a single one. > > > > But, merging both or not. > > They're different logical changes to the source code -- one refactors to > use a local variable, the other adds extra checks. I'm fine with it either way :) > > --D > > > Reviewed-by: Carlos Maiolino <cmaiolino@xxxxxxxxxx> > > > > > --- > > > fs/xfs/libxfs/xfs_da_btree.c | 4 +++- > > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > > > > > > diff --git a/fs/xfs/libxfs/xfs_da_btree.c b/fs/xfs/libxfs/xfs_da_btree.c > > > index 8ffe17b223da..376bee94b5dd 100644 > > > --- a/fs/xfs/libxfs/xfs_da_btree.c > > > +++ b/fs/xfs/libxfs/xfs_da_btree.c > > > @@ -1523,8 +1523,10 @@ xfs_da3_node_lookup_int( > > > break; > > > } > > > > > > - blk->magic = XFS_DA_NODE_MAGIC; > > > + if (magic != XFS_DA_NODE_MAGIC && magic != XFS_DA3_NODE_MAGIC) > > > + return -EFSCORRUPTED; > > > > > > + blk->magic = XFS_DA_NODE_MAGIC; > > > > > > /* > > > * Search an intermediate node for a match. > > > > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > -- > > Carlos > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Carlos -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
