Project

General

Profile

Actions

Bug #19059

closed

rgw_file: restore (corrected) fix for dir "partial match" (return of FLAG_EXACT_MATCH)

Added by Matt Benjamin about 7 years ago. Updated over 6 years ago.

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

0%

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

Description

Allow callers of rgw_lookup() on objects attested in an rgw_readdir() callback the ability to bypass exact match in RGWLibFS::stat_leaf() case 2, but restore exact match enforcement for general lookups.

This preserves required common_prefix namespace behavior, but prevents clients from eerily permitting things like "cd sara0" via partial name match on "sara01." With the original common_prefix name fix and namespace invalidates, this behavior was "somewhat harmless," in that creating objects in the non-existent directory had the effect of creating it. Still, nobody expects "cd" to create directories, do they?


Related issues 2 (0 open2 closed)

Copied to rgw - Backport #19152: jewel: rgw_file: restore (corrected) fix for dir "partial match" (return of FLAG_EXACT_MATCH)ResolvedNathan CutlerActions
Copied to rgw - Backport #20195: kraken: rgw_file: restore (corrected) fix for dir "partial match" (return of FLAG_EXACT_MATCH)ResolvedNathan CutlerActions
Actions #1

Updated by Matt Benjamin about 7 years ago

  • Status changed from In Progress to Pending Backport
Actions #2

Updated by Nathan Cutler about 7 years ago

  • Copied to Backport #19152: jewel: rgw_file: restore (corrected) fix for dir "partial match" (return of FLAG_EXACT_MATCH) added
Actions #3

Updated by Nathan Cutler almost 7 years ago

  • Backport changed from jewel to jewel,kraken
Actions #4

Updated by Nathan Cutler almost 7 years ago

  • Copied to Backport #20195: kraken: rgw_file: restore (corrected) fix for dir "partial match" (return of FLAG_EXACT_MATCH) added
Actions #5

Updated by Nathan Cutler over 6 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF