Project

General

Profile

Bug #41429

Incorrect logical operator in Monitor::handle_auth_request()

Added by yupeng chen 24 days ago. Updated 13 days ago.

Status:
Pending Backport
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
Due date:
% Done:

0%

Source:
Community (user)
Tags:
Backport:
nautilus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Monitor
Pull request ID:

Description

When checking auth_mode against AUTH_MODE_MON and AUTH_MODE_MON_MAX in Monitor::handle_auth_request(),
a logical AND (&&) operator produces a contradiction.


Related issues

Copied to RADOS - Backport #41705: nautilus: Incorrect logical operator in Monitor::handle_auth_request() New

History

#1 Updated by yupeng chen 24 days ago

“&&” in the following code snippet:

  } else if (auth_meta->auth_mode < AUTH_MODE_MON &&
         auth_meta->auth_mode > AUTH_MODE_MON_MAX) {
    derr << __func__ << " unrecognized auth mode " << auth_meta->auth_mode
     << dendl;
    return -EACCES;
  }

#2 Updated by Patrick Donnelly 23 days ago

  • Project changed from Ceph to RADOS
  • Category deleted (Monitor)
  • Target version changed from v14.2.2 to v15.0.0
  • Start date deleted (08/26/2019)
  • Backport set to nautilus,mimic
  • Affected Versions deleted (v14.2.0, v14.2.1, v14.2.2, v15.0.0)
  • Component(RADOS) Monitor added

#3 Updated by Neha Ojha 21 days ago

  • Assignee set to Neha Ojha
  • Backport deleted (nautilus,mimic)

#4 Updated by Neha Ojha 21 days ago

  • Status changed from New to Need Review
  • Backport set to nautilus
  • Pull request ID set to 30015

#5 Updated by Kefu Chai 13 days ago

  • Status changed from Need Review to Pending Backport

#6 Updated by Kefu Chai 13 days ago

  • Assignee deleted (Neha Ojha)

#7 Updated by Nathan Cutler 12 days ago

  • Copied to Backport #41705: nautilus: Incorrect logical operator in Monitor::handle_auth_request() added

Also available in: Atom PDF