]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
tools/rados: assign to optional<> without deref'ing it 39329/head
authorKefu Chai <kchai@redhat.com>
Wed, 24 Feb 2021 14:24:50 +0000 (22:24 +0800)
committerRadoslaw Zarzynski <rzarzyns@redhat.com>
Wed, 24 Feb 2021 14:37:59 +0000 (14:37 +0000)
this change addresses a regression introduced by
d333b35aa10bf03a8bc047994d5cf3fed019b49a

Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 41d68594ae6df343782a1c382ac0bddc8cb7cb69)

src/tools/rados/rados.cc

index af69c73759e9c84d22eb920100394691d92af5ef..280a51dddb8c007222d619bdd56998b63981eb90 100644 (file)
@@ -2705,7 +2705,7 @@ static int rados_tool_common(const std::map < std::string, std::string > &opts,
     }
     string attr_name(nargs[obj_name ? 1 : 2]);
     if (!obj_name) {
-      *obj_name = nargs[1];
+      obj_name = nargs[1];
     }
     bufferlist bl;
     ret = detail::getxattr(io_ctx, *obj_name, attr_name, bl, use_striper);
@@ -2726,7 +2726,7 @@ static int rados_tool_common(const std::map < std::string, std::string > &opts,
 
     string attr_name(nargs[obj_name ? 1 : 2]);
     if (!obj_name) {
-      *obj_name = nargs[1];
+      obj_name = nargs[1];
     }
     ret = detail::rmxattr(io_ctx, *obj_name, attr_name, use_striper);