We are vmalloc()''ing the fspath now so we should be vfree''ing
it and not
kfreeing it. Thanks,
Reported-by: Dave Sterba <dave@jikos.cz>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
---
fs/btrfs/backref.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c
index fb72931..395d912 100644
--- a/fs/btrfs/backref.c
+++ b/fs/btrfs/backref.c
@@ -1634,6 +1634,6 @@ void free_ipath(struct inode_fs_paths *ipath)
{
if (!ipath)
return;
- kfree(ipath->fspath);
+ vfree(ipath->fspath);
kfree(ipath);
}
--
1.7.7.6
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Sep 14, 2012 at 01:05:09PM -0400, Josef Bacik wrote:> We are vmalloc()''ing the fspath now so we should be vfree''ing it and not > kfreeing it. Thanks, > > Reported-by: Dave Sterba <dave@jikos.cz> > Signed-off-by: Josef Bacik <jbacik@fusionio.com>IMO better to fold this change into the patch that introduced it, as it''s not merged anywhere else than btrfs-next: [PATCH v2 6/6] Btrfs: use larger limit for translation of logical to inode From: Liu Bo <bo.li.liu@oracle.com> plus add the missing #include <linux/vmalloc.h> that Fengguang Wu encountered. david -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html