diff options
author | Eric Sandeen <sandeen@sandeen.net> | 2009-10-30 09:27:07 +0100 |
---|---|---|
committer | hch@lst.de <Christoph Hellwig> | 2009-10-30 09:27:07 +0100 |
commit | 3b826386d376e5545d2e92b2da5ebd965cafae97 (patch) | |
tree | 50c017ab5f90a70878291b33c331004b36437cce /fs/xfs/xfs_iomap.c | |
parent | Merge branch 'master' of ssh://oss.sgi.com/oss/git/xfs/xfs into for-linus (diff) | |
download | linux-3b826386d376e5545d2e92b2da5ebd965cafae97.tar.xz linux-3b826386d376e5545d2e92b2da5ebd965cafae97.zip |
xfs: free temporary cursor in xfs_dialloc
Commit bd169565993b39b9b4b102cdac8b13e0a259ce2f seems
to have a slight regression where this code path:
if (!--searchdistance) {
/*
* Not in range - save last search
* location and allocate a new inode
*/
...
goto newino;
}
doesn't free the temporary cursor (tcur) that got dup'd in
this function.
This leaks an item in the xfs_btree_cur zone, and it's caught
on module unload:
===========================================================
BUG xfs_btree_cur: Objects remaining on kmem_cache_close()
-----------------------------------------------------------
It seems like maybe a single free at the end of the function might
be cleaner, but for now put a del_cursor right in this code block
similar to the handling in the rest of the function.
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs/xfs/xfs_iomap.c')
0 files changed, 0 insertions, 0 deletions