From: Ilya Dryomov <idryomov@gmail.com>
Date: Fri, 24 Nov 2023 18:53:48 +0000 (+0100)
Subject: tools/rados: just return instead of exit(1) in "rados cppool" handler
X-Git-Tag: v19.3.0~314^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=de6635582b7615eb54800a616711073a3c7b0dbd;p=ceph-ci.git

tools/rados: just return instead of exit(1) in "rados cppool" handler

Otherwise an occasional segfault occurs.  This instance was missed in
commit 2c149262888c ("tools/rados: always call rados.shutdown() before
exit()").

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
---

diff --git a/src/tools/rados/rados.cc b/src/tools/rados/rados.cc
index db3aa543fb5..4a38d33abd2 100644
--- a/src/tools/rados/rados.cc
+++ b/src/tools/rados/rados.cc
@@ -3133,7 +3133,7 @@ static int rados_tool_common(const std::map < std::string, std::string > &opts,
 	cerr << "    If you insist on making a broken copy, you can pass\n"
 	     << "    --yes-i-really-mean-it to proceed anyway."
 	     << std::endl;
-	exit(1);
+	return 1;
       }
     }