Project

General

Profile

Actions
Copy link

Bug #10458

closed

librados: original pool name cached in IoCtxImpl may be used after pool is renamed

Added by Josh Durgin over 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
High
Assignee:
Sage Weil
Category:
librados
Target version:
-
% Done:

0%

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

Description

This isn't a big deal since (1) pool renames are rare and (2) it's only used in a few methods:

librados::IoCtx::get_pool_name() const
rados_ioctx_pool_stat()
rados_ioctx_get_pool_name()

We should probably just get rid of the IoCtxImpl::pool_name member, and always look it up when it's needed.

Files

20150122_10458_pool-name-caching-fix_v1.patch (6.29 KB) 20150122_10458_pool-name-caching-fix_v1.patch Radoslaw Zarzynski, 01/23/2015 01:31 PM
Actions
Copy link
 #1

Updated by Radoslaw Zarzynski about 9 years ago

I am working on this bug.

Actions
Copy link
 #3

Updated by Radoslaw Zarzynski about 9 years ago

Initial proposal of the patch in attachment.

Actions
Copy link
 #4

Updated by Sage Weil about 9 years ago

  • Assignee set to Sage Weil
  • Priority changed from Normal to High
Actions
Copy link
 #5

Updated by Sage Weil about 9 years ago

  • Status changed from New to Fix Under Review
Actions
Copy link
 #7

Updated by Samuel Just about 9 years ago

  • Status changed from Fix Under Review to Resolved
Actions
Copy link

Also available in: Atom PDF