Project

General

Profile

Actions

Bug #57325

closed

memory issues in newDBStore()

Added by Yuval Lifshitz over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

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

Description

in: rgw_sal_dbstore.cc

1. unless we have an overloaded "new" operator for rgw::sal::DBStore, we should not check for successful allocations (we should always crash on OOM instead)
2. we dereference "db" even if we get null values from: dbsm->getDB()
3. we may pass freed memory of "dbsm" to setDBStoreManager()

coverity CID 1511420

Actions #1

Updated by Soumya Koduri over 1 year ago

  • Assignee set to Soumya Koduri
Actions #2

Updated by Soumya Koduri over 1 year ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 47970
Actions #3

Updated by Casey Bodley over 1 year ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF