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

fs: segfault launching ceph-fuse with bad --name #5382

Merged
1 commit merged into from Sep 6, 2015

Conversation

Projects
None yet
2 participants
@ghost
Copy link

commented Jul 28, 2015

auth: check return value of keyring->get_secret
get_secret can fail to populate the passed CryptoKey, for
example if the entity name is not found in the keyring.  In
this case, attempts to use the CryptoKey will lead to
segfaults.

Fixes: #12417
Signed-off-by: John Spray <john.spray@redhat.com>
(cherry picked from commit 64e5041)

@ghost ghost self-assigned this Jul 28, 2015

@ghost ghost added this to the hammer milestone Jul 28, 2015

@ghost ghost added bug fix cephfs labels Jul 28, 2015

ghost pushed a commit that referenced this pull request Jul 29, 2015

@ghost

This comment has been minimized.

Copy link
Author

commented Aug 30, 2015

@liewegas does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/12701#rados for details ).

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5382: segfault launching ceph-fuse with bad --name
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5382: segfault launching ceph-fuse with bad --name
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5382: segfault launching ceph-fuse with bad --name
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5382: segfault launching ceph-fuse with bad --name
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 1, 2015

Merge pull request #5382: segfault launching ceph-fuse with bad --name
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5382: segfault launching ceph-fuse with bad --name
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost

This comment has been minimized.

Copy link
Author

commented Sep 6, 2015

@liewegas ping ?

@liewegas

This comment has been minimized.

Copy link
Member

commented Sep 6, 2015

👍

ghost pushed a commit that referenced this pull request Sep 6, 2015

Loic Dachary
Merge pull request #5382 from dachary/wip-12500-hammer
segfault launching ceph-fuse with bad --name

Reviewed-by: Sage Weil <sage@redhat.com>

@ghost ghost merged commit 47af509 into ceph:hammer Sep 6, 2015

@ghost ghost changed the title segfault launching ceph-fuse with bad --name fs: segfault launching ceph-fuse with bad --name Sep 13, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.