Skip to content

Commit d32c879

Browse files
authored
Merge pull request ceph#59028 from cbodley/wip-67326
rgw/notify: visit() returns copy of owner string Reviewed-by: Adam Emerson <[email protected]>
2 parents 52d63b5 + fe4f03d commit d32c879

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/rgw/driver/rados/rgw_notify.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1111,8 +1111,8 @@ int publish_reserve(const DoutPrefixProvider* dpp,
11111111

11121112
// reload the topic in case it changed since the notification was added
11131113
const std::string& topic_tenant = std::visit(fu2::overload(
1114-
[] (const rgw_user& u) -> const std::string& { return u.tenant; },
1115-
[] (const rgw_account_id& a) -> const std::string& { return a; }
1114+
[] (const rgw_user& u) -> std::string { return u.tenant; },
1115+
[] (const rgw_account_id& a) -> std::string { return a; }
11161116
), topic_cfg.owner);
11171117
const RGWPubSub ps(res.store, topic_tenant, site);
11181118
int ret = ps.get_topic(res.dpp, topic_cfg.dest.arn_topic,

0 commit comments

Comments
 (0)