Re: [PATCH] xfs: handle error if xfs_btree_get_bufs fails

["Darrick J. Wong" <darrick.wong@oracle.com>]

On Tue, Oct 17, 2017 at 08:27:28PM +0200, Luis R. Rodriguez wrote:
> On Fri, Oct 13, 2017 at 02:02:16AM +0200, Luis R. Rodriguez wrote:
> > On Mon, Oct 09, 2017 at 09:07:36PM -0700, Darrick J. Wong wrote:
> > > On Mon, Oct 09, 2017 at 10:07:12PM -0500, Eric Sandeen wrote:
> > > > 
> > > > 
> > > > On 10/9/17 9:19 PM, Luis R. Rodriguez wrote:
> > > > > What ever happened to this patch [0]? It was ACKed [1] 
> > > > 
> > > > (it was reviewed-by hch)
> > > > 
> > > > > but I haven't
> > > > > seen it merged. Did it fall through the cracks?
> > > > 
> > > > Seems like it, possibly because I sent V2 as a re:, accidentally.
> > > 
> > > <grumble>
> > > 
> > > Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
> > 
> > If we're going to add this shouldn't we then also have checks for
> > xfs_btree_get_bufl() calls?
> 
> *re-poke*

AFAICT the two callers of xfs_btree_get_bufl already checked that the
incoming argument isn't NULLFSBLOCK, and (at least in theory) the bnobt
shouldn't have freespace records extending outside the filesystem.

That said, new patches accepted. ;)

--D

> 
>   Luis