Skip to content

Commit

Permalink
tools/rados: just return instead of exit(1) in "rados cppool" handler
Browse files Browse the repository at this point in the history
Otherwise an occasional segfault occurs.  This instance was missed in
commit 2c14926 ("tools/rados: always call rados.shutdown() before
exit()").

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit de66355)
  • Loading branch information
idryomov authored and rishabh-d-dave committed Jan 29, 2024
1 parent e64819f commit 04dc1ca
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/tools/rados/rados.cc
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
}

Expand Down

0 comments on commit 04dc1ca

Please sign in to comment.