XFS: Free buffer pages array unconditionally
authorDave Chinner <david@fromorbit.com>
Mon, 14 Dec 2009 23:11:57 +0000 (23:11 +0000)
committerAlex Elder <aelder@sgi.com>
Wed, 16 Dec 2009 19:41:20 +0000 (13:41 -0600)
commit3fc98b1ac036675b95f6e3fafd5ef147b97d4d30
tree1710b6e0140a12a6e20ef0494b11e42b9f0fcfc6
parenta5f9be58c2b87106100a6053d09b1f9f8d551c6e
XFS: Free buffer pages array unconditionally

The code in xfs_free_buf() only attempts to free the b_pages array if the
buffer is a page cache backed or page allocated buffer. The extra log buffer
that is used when the log wraps uses pages that are allocated to a different
log buffer, but it still has a b_pages array allocated when those pages
are associated to with the extra buffer in xfs_buf_associate_memory.

Hence we need to always attempt to free the b_pages array when tearing
down a buffer, not just on buffers that are explicitly marked as page bearing
buffers. This fixes a leak detected by the kernel memory leak code.

Signed-off-by: Dave Chinner <david@fromorbit.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
fs/xfs/linux-2.6/xfs_buf.c