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@xxxxxxxxxx> > > > > 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 -- 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
