Project

General

Profile

Actions
Copy link

Bug #49173

closed

C++ write API might not properly handle oversized bufferlist

Added by Jason Dillaman about 3 years ago. Updated about 3 years ago.

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

0%

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

Description

The C++ API takes an offset, length, and bufferlist. The semi-recent change to the ImageRequest state machine optimized away the need to track the length assuming it would always match the provided bufferlist length. This holds true for the C API but not necessarily the C++ API. This can result in an unexpectedly large write operation.

Related issues 2 (0 open — 2 closed)

Copied to rbd - Backport #49198: pacific: C++ write API might not properly handle oversized bufferlistResolvedJason DillamanActions
Copied to rbd - Backport #49199: octopus: C++ write API might not properly handle oversized bufferlistResolvedNathan CutlerActions
Actions
Copy link
 #1

Updated by Jason Dillaman about 3 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 39303
Actions
Copy link
 #2

Updated by Mykola Golub about 3 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions
Copy link
 #3

Updated by Backport Bot about 3 years ago

  • Copied to Backport #49198: pacific: C++ write API might not properly handle oversized bufferlist added
Actions
Copy link
 #4

Updated by Backport Bot about 3 years ago

  • Copied to Backport #49199: octopus: C++ write API might not properly handle oversized bufferlist added
Actions
Copy link
 #5

Updated by Loïc Dachary about 3 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions
Copy link

Also available in: Atom PDF