On 9/11/13 4:17 PM, Dan Carpenter wrote:
> There is a '=' vs '==' typo so the ASSERT()s are always true.
And worse, it assigns to di_format on a debug kernel, eek,
although should be modifying it to the thing it's supposed
to be anyway. ;) It's a self-healing ASSERT! :)
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> ---
> Introduced in 21b5c9784b ('xfs: swap extents operations for CRC
> filesystems')
>
> diff --git a/fs/xfs/xfs_bmap_btree.c b/fs/xfs/xfs_bmap_btree.c
> index 531b020..bb8de8e 100644
> --- a/fs/xfs/xfs_bmap_btree.c
> +++ b/fs/xfs/xfs_bmap_btree.c
> @@ -957,9 +957,9 @@ xfs_bmbt_change_owner(
> ASSERT(tp || buffer_list);
> ASSERT(!(tp && buffer_list));
> if (whichfork == XFS_DATA_FORK)
> - ASSERT(ip->i_d.di_format = XFS_DINODE_FMT_BTREE);
> + ASSERT(ip->i_d.di_format == XFS_DINODE_FMT_BTREE);
> else
> - ASSERT(ip->i_d.di_aformat = XFS_DINODE_FMT_BTREE);
> + ASSERT(ip->i_d.di_aformat == XFS_DINODE_FMT_BTREE);
>
> cur = xfs_bmbt_init_cursor(ip->i_mount, tp, ip, whichfork);
> if (!cur)
>
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
