The atoi exception was not caught

[Ranxy]

Search before asking

• I had searched in the issues and found no similar issues.

Motivation

There is some code in the project as follows

try {
        slot_ = atoi(args[2].c_str());
        dst_node_id_ = args[3];
      } catch (std::exception &e) {
        return Status(Status::RedisParseErr, errValueNotInterger);
      }

However atoi does not throw exceptions, so perhaps we should use std::stoi to replace these.

Solution

Replace atoi to std::stoi

Are you willing to submit a PR?

• I'm willing to submit a PR!

[git-hulk]

Good catch, I think we can use Util::OctalStringToNum to replace the atoi which also can check the min/max boundary.