Bug #13424
wrong conditional for boolean function KeyServer::get_auth()
% Done:
0%
Source:
Community (dev)
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
In src/auth/cephx/CephxServiceHandler.cc
there is a conditional:
if (key_server->get_auth(entity_name, eauth) < 0) { ret = -EPERM; break; }
As stated in #9756 this conditional will never fire because the function in question, KeyServer::get_auth()
returns a boolean value that by definition will never be less than zero.
Related issues
History
#1 Updated by Nathan Cutler over 8 years ago
infernalis PR: https://github.com/ceph/ceph/pull/6211
#2 Updated by Nathan Cutler over 8 years ago
- Status changed from Pending Backport to Resolved