mds: add a proper check to evict clients with id 0

Fixes: https://tracker.ceph.com/issues/70198
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>

diff --git a/src/mds/SessionMap.cc b/src/mds/SessionMap.cc
index 95daccac2ec68287c6fb1cef31bbad7b22850404..3aa3969d2943ed77c16d23c0248bae66a00ebba7 100644 (file)
--- a/src/mds/SessionMap.cc
+++ b/src/mds/SessionMap.cc
@@ -1303,10 +1303,16 @@ int SessionFilter::parse(
       if (v == "*") {
         // evict all clients , by default id set to 0
         return 0;
-      } else if (v == "0") {
-        *ss << "Invalid value";
-        return -EINVAL;
-      }
+      } else {
+        try {
+          if (!std::stoi(v)) {
+            *ss << "Invalid value";
+            return -EINVAL; } 
+          } catch (...) {
+            *ss << "Invalid input";
+            return -EINVAL;
+          }
+        }
       id = strict_strtoll(v.c_str(), 10, &err);
       if (!err.empty()) {
         *ss << err;