Add input signature_script checking to submitTransaction RPC

rpc/core/src/error.rs

@@ -1,4 +1,4 @@
-use kaspa_consensus_core::{subnets::SubnetworkConversionError, tx::TransactionId};
+use kaspa_consensus_core::{subnets::SubnetworkConversionError, tx::TransactionId};
 use kaspa_utils::networking::IpAddress;
 use std::{net::AddrParseError, num::TryFromIntError};
 use thiserror::Error;
@@ -47,6 +47,9 @@ pub enum RpcError {
     #[error("Rejected transaction {0}: {1}")]
     RejectedTransaction(RpcTransactionId, String),
 
+    #[error("Transaction {0} has an input with an empty signature_script at index {1}.")]
+    EmptySignatureScript(RpcTransactionId, usize),
+
     #[error("Block {0} is invalid. No verbose data can be built.")]
     InvalidBlock(RpcHash),
 
@@ -116,9 +119,9 @@ pub enum RpcError {
     #[error("transaction query must either not filter transactions or include orphans")]
     InconsistentMempoolTxQuery,
 
-    #[error(transparent)]
-    SubnetParsingError(#[from] SubnetworkConversionError),
-
+    #[error(transparent)]
+    SubnetParsingError(#[from] SubnetworkConversionError),
+
     #[error(transparent)]
     WasmError(#[from] workflow_wasm::error::Error),
 

rpc/service/src/service.rs

@@ -493,6 +493,13 @@ NOTE: This error usually indicates an RPC conversion error between the node and
 
         let transaction: Transaction = (&request.transaction).try_into()?;
         let transaction_id = transaction.id();
+        let inputs = &transaction.inputs;
+
+        if let Some((index, _)) = inputs.iter().enumerate().find(|(_, input)| input.signature_script.is_empty()) {
+            let err = RpcError::EmptySignatureScript(transaction_id, index);
+            return Err(err)
+        }
+
         let session = self.consensus_manager.consensus().unguarded_session();
         let orphan = match allow_orphan {
             true => Orphan::Allowed,