New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

client: fix Client::handle_cap_flushsnap_ack() crash #12859

Merged
merged 1 commit into from Jan 13, 2017

Conversation

Projects
None yet
3 participants
@ukernel
Member

ukernel commented Jan 10, 2017

No description provided.

client: fix Client::handle_cap_flushsnap_ack() crash
Struct CapSnap holds a reference to its parent inode. So erasing
struct CapSnap from Inode::cap_snaps may drop inode's last reference.
The inode gets freed in the middle of erasing struct CapSnap

Fixes: http://tracker.ceph.com/issues/18460
Signed-off-by: Yan, Zheng <zyan@redhat.com>

@ukernel ukernel changed the title from Wip 18460 to client: fix Client::handle_cap_flushsnap_ack() crash Jan 10, 2017

@gregsfortytwo

LGTM

@jcsp jcsp merged commit 868d419 into ceph:master Jan 13, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@ukernel ukernel deleted the ukernel:wip-18460 branch Feb 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment