Fix ledger_entry

oleks-rip · Nov 21, 2024 · 38f9a60 · 38f9a60
1 parent cf6281b
commit 38f9a60
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/include/xrpl/protocol/detail/features.macro b/include/xrpl/protocol/detail/features.macro
@@ -29,6 +29,7 @@
 // If you add an amendment here, then do not forget to increment `numFeatures`
 // in include/xrpl/protocol/Feature.h.
 
+XRPL_FEATURE(PermissionedDomains,        Supported::no,  VoteBehavior::DefaultNo)
 XRPL_FEATURE(Credentials,                Supported::yes, VoteBehavior::DefaultNo)
 XRPL_FEATURE(AMMClawback,                Supported::yes, VoteBehavior::DefaultNo)
 XRPL_FIX    (AMMv1_2,                    Supported::yes, VoteBehavior::DefaultNo)
@@ -98,8 +99,6 @@ XRPL_FIX    (1513,                       Supported::yes, VoteBehavior::DefaultYe
 XRPL_FEATURE(FlowCross,                  Supported::yes, VoteBehavior::DefaultYes)
 XRPL_FEATURE(Flow,                       Supported::yes, VoteBehavior::DefaultYes)
 XRPL_FEATURE(OwnerPaysFee,               Supported::no,  VoteBehavior::DefaultNo)
-XRPL_FEATURE(PermissionedDomains,        Supported::no,  VoteBehavior::DefaultNo)
-
 
 // The following amendments are obsolete, but must remain supported
 // because they could potentially get enabled.

diff --git a/src/xrpld/rpc/handlers/LedgerEntry.cpp b/src/xrpld/rpc/handlers/LedgerEntry.cpp
@@ -809,6 +809,7 @@ doLedgerEntry(RPC::JsonContext& context)
         }
         else if (context.params.isMember(jss::permissioned_domain))
         {
+            expectedType = ltPERMISSIONED_DOMAIN;
             uNodeIndex = beast::zero;
             auto const& pd = context.params[jss::permissioned_domain];
             if (pd.isString())
@@ -822,6 +823,7 @@ doLedgerEntry(RPC::JsonContext& context)
             else if (
                 !pd.isObject() || !pd.isMember(jss::account) ||
                 !pd[jss::account].isString() || !pd.isMember(jss::seq) ||
+                (pd[jss::seq].isInt() && pd[jss::seq].asInt() < 0) ||
                 (!pd[jss::seq].isInt() && !pd[jss::seq].isUInt()))
             {
                 jvResult[jss::error] = "malformedRequest";
@@ -905,6 +907,10 @@ doLedgerEntry(RPC::JsonContext& context)
             jvResult[jss::index] = to_string(uNodeIndex);
         }
     }
+    else
+    {
+        jvResult[jss::error] = "malformedRequest";
+    }
 
     return jvResult;
 }