refactor: improve edge case handling for recursion limits

CHANGELOG.md

@@ -52,6 +52,7 @@ Every module contains its own CHANGELOG.md. Please refer to the module you are i
 * (testutil/integration) [#22616](https://github.com/cosmos/cosmos-sdk/pull/22616) Remove double context in integration tests v1.
     * Use integrationApp.Context() instead of creating a context prior.
 * [#22826](https://github.com/cosmos/cosmos-sdk/pull/22826) Simplify testing frameworks by removing `testutil/cmdtest`.
+* [#22988](https://github.com/cosmos/cosmos-sdk/pull/22988) Improve edge case handling for maxDepth and maxCalls.
 
 ### Bug Fixes
 

codec/types/interface_registry.go

@@ -274,10 +274,10 @@ func (r statefulUnpacker) cloneForRecursion() *statefulUnpacker {
 // UnpackAny deserializes a protobuf Any message into the provided interface, ensuring the interface is a pointer.
 // It applies stateful constraints such as max depth and call limits, and unpacks interfaces if required.
 func (r *statefulUnpacker) UnpackAny(any *Any, iface interface{}) error {
-	if r.maxDepth == 0 {
+	if r.maxDepth <= 0 {
 		return errors.New("max depth exceeded")
 	}
-	if r.maxCalls.count == 0 {
+	if r.maxCalls.count <= 0 {
 		return errors.New("call limit exceeded")
 	}
 	// here we gracefully handle the case in which `any` itself is `nil`, which may occur in message decoding

codec/unknownproto/unknown_fields.go

@@ -54,7 +54,7 @@ func doRejectUnknownFields(
 	if len(bz) == 0 {
 		return hasUnknownNonCriticals, nil
 	}
-	if recursionLimit == 0 {
+	if recursionLimit <= 0 {
 		return false, errors.New("recursion limit reached")
 	}