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

osdc/Objecter: release message if it's not handled #15890

Merged
merged 2 commits into from Jun 25, 2017

Conversation

Projects
None yet
3 participants
@tchaikov
Copy link
Contributor

tchaikov commented Jun 23, 2017

Fixes: http://tracker.ceph.com/issues/19741
Signed-off-by: Kefu Chai kchai@redhat.com

tchaikov added some commits Jun 21, 2017

crypto: allow PK11 module to load even if it's already initialized
there is chance that other pieces of application loads PK11 module
already and does not finalize it before calling common_init_finish().

also, upon fork, PK11 module resets its entire status including `nsc_init`,
by which PK11 module tell if it is initialized or not. so the behavior
of NSS_InitContext() could be different before and after fork. that's
another reason to ignore CKR_CRYPTOKI_ALREADY_INITIALIZED error (see
NSS_GetError()).

Fixes: http://tracker.ceph.com/issues/19741
Signed-off-by: Kefu Chai <kchai@redhat.com>
osdc/Objecter: release message if it is not handled
Fixes: http://tracker.ceph.com/issues/19741
Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov added the bug fix label Jun 23, 2017

@tchaikov tchaikov requested review from liewegas and jdurgin Jun 23, 2017

@tchaikov tchaikov added the needs-qa label Jun 23, 2017

@jdurgin jdurgin added the common label Jun 23, 2017

@liewegas liewegas merged commit 82fcdd9 into ceph:master Jun 25, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
make check make check succeeded
Details

@tchaikov tchaikov deleted the tchaikov:wip-19741 branch Jun 25, 2017

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.