summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSage Weil <sage@newdream.net>2010-04-29 09:28:11 -0700
committerSage Weil <sage@newdream.net>2010-05-17 15:25:25 -0700
commita6424e48c8d54a5795430b07c4487f1ed280df4e (patch)
tree34731c9cb1df3cfe885d4526ce79ca50c50b7b4a
parent9dd4658db1be5ca92c2ed2fd7a100d973125d9c5 (diff)
downloadlinux-stable-a6424e48c8d54a5795430b07c4487f1ed280df4e.tar.gz
linux-stable-a6424e48c8d54a5795430b07c4487f1ed280df4e.tar.bz2
linux-stable-a6424e48c8d54a5795430b07c4487f1ed280df4e.zip
ceph: fix xattr dangling pointer / double free
If we use the xattr_blob, clear the pointer so we don't release the memory at the bottom of the fuction. Reported-by: Henry C Chang <henry_c_chang@tcloudcomputing.com> Signed-off-by: Sage Weil <sage@newdream.net>
-rw-r--r--fs/ceph/inode.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c
index ef917232cf37..913cafd70cd0 100644
--- a/fs/ceph/inode.c
+++ b/fs/ceph/inode.c
@@ -619,6 +619,7 @@ static int fill_inode(struct inode *inode,
memcpy(ci->i_xattrs.blob->vec.iov_base,
iinfo->xattr_data, iinfo->xattr_len);
ci->i_xattrs.version = le64_to_cpu(info->xattr_version);
+ xattr_blob = NULL;
}
inode->i_mapping->a_ops = &ceph_aops;