]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
Tools/rados: Improve Error Messaging for Object Name Resolution 55598/head
authorNitzanMordhai <nmordech@redhat.com>
Thu, 16 Nov 2023 07:09:29 +0000 (07:09 +0000)
committernmordech@redhat.com <nmordech@redhat.com>
Thu, 15 Feb 2024 10:48:40 +0000 (10:48 +0000)
The current implementation of 'rados clearomap' exhibits a behavior where
an error message is generated without the associated object name or,
in the case of a non-existent object name, may result in a segmentation fault.

The proposed fix addresses this issue by enhancing the error message.
After applying the fix, error messages will consistently display the correct
object name, providing users with more accurate and actionable information.

Fixes: https://tracker.ceph.com/issues/63541
Signed-off-by: Nitzan Mordechai <nmordech@redhat.com>
(cherry picked from commit 313bfca056ef7f83c0e8c70a3bc4ff9d070b63d9)

src/tools/rados/rados.cc

index c419a43447af18f76a6e3bdb043716adf9aa7611..4078d38bd0a10133841167313b87e18b01328fe3 100644 (file)
@@ -2971,7 +2971,7 @@ static int rados_tool_common(const std::map < std::string, std::string > &opts,
     for (const auto& oid : oids) {
       ret = io_ctx.omap_clear(oid);
       if (ret < 0) {
-        cerr << "error clearing omap keys " << pool_name << "/" << prettify(*obj_name) << "/"
+        cerr << "error clearing omap keys " << pool_name << "/" << prettify(oid) << "/"
              << cpp_strerror(ret) << std::endl;
         return 1;
       }