| Message ID | 147200548364.15538.8448801693088924431.stgit@birch.djwong.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
diff --git a/libxfs/xfs_btree.c b/libxfs/xfs_btree.c index b69c8d2..8e928ea 100644 --- a/libxfs/xfs_btree.c +++ b/libxfs/xfs_btree.c @@ -4558,6 +4558,13 @@ xfs_btree_simple_query_range( if (error) goto out; + /* Nothing? See if there's anything to the right. */ + if (!stat) { + error = xfs_btree_increment(cur, 0, &stat); + if (error) + goto out; + } + while (stat) { /* Find the record. */ error = xfs_btree_get_rec(cur, &recp, &stat);
If the initial LOOKUP_LE in the simple query range fails to find anything, we should attempt to increment the btree cursor to see if there actually /are/ records for what we're trying to find. Without this patch, a bnobt range query of (0, $agsize) returns no results because the leftmost record never has a startblock of zero. Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> --- libxfs/xfs_btree.c | 7 +++++++ 1 file changed, 7 insertions(+)