Project

General

Profile

Actions
Copy link

Bug #54452

open

Inverted check in RGWRados::check_disk_state()

Added by Daniel Gryniewicz about 2 years ago. Updated almost 2 years ago.

Status:
Pending Backport
Priority:
Normal
Assignee:
Daniel Gryniewicz
Target version:
-
% Done:

0%

Source:
Tags:
backport_processed
Backport:
quincy
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
45225
Crash signature (v1):
Crash signature (v2):

Description

This code is inverted:

const bool head_pool_found =
    get_obj_head_ioctx(dpp, bucket_info, obj, &head_obj_ctx);
  if (head_pool_found) {

This is because get_obj_head_ioctx() returns the standard int, where 0 is success. Casting to bool inverts this.

Related issues 1 (1 open — 0 closed)

Copied to rgw - Backport #54608: quincy: Inverted check in RGWRados::check_disk_state()NewActions
Actions
Copy link

Also available in: Atom PDF