[fix][sdk]Add range checking

LiuRuoyu01 · ketor · commit ee3b743ad568 · 2025-08-19T04:05:02.000Z
diff --git a/src/coordinator/coordinator_control_coor.cc b/src/coordinator/coordinator_control_coor.cc
@@ -5842,6 +5842,11 @@ butil::Status CoordinatorControl::ScanRegions(const std::string& start_key, cons
         continue;
       }
     }
+    CHECK(upper_bound > region_internal.definition().range().start_key())
+        << fmt::format("end_key should greater than region_start_key, range: [{}, {}], region_range: [{}, {}]",
+                       Helper::StringToHex(lower_bound), Helper::StringToHex(upper_bound),
+                       Helper::StringToHex(region_internal.definition().range().start_key()),
+                       Helper::StringToHex(region_internal.definition().range().end_key()));
 
     regions.push_back(region_internal);
 

Original file line number	Diff line number	Diff line change
`@@ -5842,6 +5842,11 @@ butil::Status CoordinatorControl::ScanRegions(const std::string& start_key, cons`
`5842`	`5842`	`continue;`
`5843`	`5843`	`}`
`5844`	`5844`	`}`
	`5845`	`+ CHECK(upper_bound > region_internal.definition().range().start_key())`
	`5846`	`+ << fmt::format("end_key should greater than region_start_key, range: [{}, {}], region_range: [{}, {}]",`
	`5847`	`+ Helper::StringToHex(lower_bound), Helper::StringToHex(upper_bound),`
	`5848`	`+ Helper::StringToHex(region_internal.definition().range().start_key()),`
	`5849`	`+ Helper::StringToHex(region_internal.definition().range().end_key()));`
`5845`	`5850`
`5846`	`5851`	`regions.push_back(region_internal);`
`5847`	`5852`