Skip to content
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: avoid creating orphan object in Client::check_pool_perm() #6603

Merged
merged 1 commit into from Nov 20, 2015

Conversation

ukernel
Copy link
Contributor

@ukernel ukernel commented Nov 16, 2015

Fixes: #13782
Signed-off-by: Yan, Zheng zyan@redhat.com

Fixes: ceph#13782
Signed-off-by: Yan, Zheng <zyan@redhat.com>
@ukernel ukernel added bug-fix cephfs Ceph File System labels Nov 16, 2015
@jcsp
Copy link
Contributor

jcsp commented Nov 20, 2015

I guess we could be smarter and only check the read side here, but it doesn't seem worth the complexity so I think this a pretty reasonable behaviour

jcsp added a commit that referenced this pull request Nov 20, 2015
client: avoid creating orphan object in Client::check_pool_perm()

Reviewed-by: John Spray <john.spray@redhat.com>
@jcsp jcsp merged commit 0b21492 into ceph:master Nov 20, 2015
@ukernel ukernel deleted the wip-13782 branch December 3, 2015 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix cephfs Ceph File System
Projects
None yet
3 participants