extract solana-bincode crate (anza-xyz#3020)

* extract limited-deserialize crate

* rename solana-limited-deserialize to solana-bincode

Cargo.toml

@@ -104,6 +104,7 @@ members = [
     "sdk/account",
     "sdk/account-info",
     "sdk/atomic-u64",
+    "sdk/bincode",
     "sdk/cargo-build-sbf",
     "sdk/cargo-test-sbf",
     "sdk/clock",
@@ -375,6 +376,7 @@ solana-banks-client = { path = "banks-client", version = "=2.1.0" }
 solana-banks-interface = { path = "banks-interface", version = "=2.1.0" }
 solana-banks-server = { path = "banks-server", version = "=2.1.0" }
 solana-bench-tps = { path = "bench-tps", version = "=2.1.0" }
+solana-bincode = { path = "sdk/bincode", version = "=2.1.0" }
 solana-bloom = { path = "bloom", version = "=2.1.0" }
 solana-bn254 = { path = "curves/bn254", version = "=2.1.0" }
 solana-bpf-loader-program = { path = "programs/bpf_loader", version = "=2.1.0" }

sdk/bincode/Cargo.toml

@@ -0,0 +1,26 @@
+[package]
+name = "solana-bincode"
+description = "Solana bincode utilities"
+documentation = "https://docs.rs/solana-bincode"
+version = { workspace = true }
+authors = { workspace = true }
+repository = { workspace = true }
+homepage = { workspace = true }
+license = { workspace = true }
+edition = { workspace = true }
+
+[dependencies]
+bincode = { workspace = true }
+serde = { workspace = true }
+solana-instruction = { workspace = true, default-features = false, features = [
+    "std",
+] }
+
+[dev-dependencies]
+solana-program = { path = "../program" }
+
+[package.metadata.docs.rs]
+targets = ["x86_64-unknown-linux-gnu"]
+
+[lints]
+workspace = true

sdk/bincode/src/lib.rs

@@ -0,0 +1,48 @@
+//! Contains a single utility function for deserializing from [bincode].
+//!
+//! [bincode]: https://docs.rs/bincode
+
+use {bincode::config::Options, solana_instruction::error::InstructionError};
+
+/// Deserialize with a limit based the maximum amount of data a program can expect to get.
+/// This function should be used in place of direct deserialization to help prevent OOM errors
+pub fn limited_deserialize<T>(instruction_data: &[u8], limit: u64) -> Result<T, InstructionError>
+where
+    T: serde::de::DeserializeOwned,
+{
+    bincode::options()
+        .with_limit(limit)
+        .with_fixint_encoding() // As per https://github.com/servo/bincode/issues/333, these two options are needed
+        .allow_trailing_bytes() // to retain the behavior of bincode::deserialize with the new `options()` method
+        .deserialize_from(instruction_data)
+        .map_err(|_| InstructionError::InvalidInstructionData)
+}
+
+#[cfg(test)]
+pub mod tests {
+    use {super::*, solana_program::system_instruction::SystemInstruction};
+
+    #[test]
+    fn test_limited_deserialize_advance_nonce_account() {
+        let item = SystemInstruction::AdvanceNonceAccount;
+        let mut serialized = bincode::serialize(&item).unwrap();
+
+        assert_eq!(
+            serialized.len(),
+            4,
+            "`SanitizedMessage::get_durable_nonce()` may need a change"
+        );
+
+        assert_eq!(
+            limited_deserialize::<SystemInstruction>(&serialized, 4).as_ref(),
+            Ok(&item)
+        );
+        assert!(limited_deserialize::<SystemInstruction>(&serialized, 3).is_err());
+
+        serialized.push(0);
+        assert_eq!(
+            limited_deserialize::<SystemInstruction>(&serialized, 4).as_ref(),
+            Ok(&item)
+        );
+    }
+}

sdk/program/Cargo.toml

@@ -33,6 +33,7 @@ sha2 = { workspace = true }
 sha3 = { workspace = true }
 solana-account-info = { workspace = true, features = ["bincode"] }
 solana-atomic-u64 = { workspace = true }
+solana-bincode = { workspace = true }
 solana-clock = { workspace = true, features = ["serde"] }
 solana-decode-error = { workspace = true }
 solana-frozen-abi = { workspace = true, optional = true, features = ["frozen-abi"] }

sdk/program/src/program_utils.rs

@@ -1,48 +1 @@
-//! Contains a single utility function for deserializing from [bincode].
-//!
-//! [bincode]: https://docs.rs/bincode
-
-use {crate::instruction::InstructionError, bincode::config::Options};
-
-/// Deserialize with a limit based the maximum amount of data a program can expect to get.
-/// This function should be used in place of direct deserialization to help prevent OOM errors
-pub fn limited_deserialize<T>(instruction_data: &[u8], limit: u64) -> Result<T, InstructionError>
-where
-    T: serde::de::DeserializeOwned,
-{
-    bincode::options()
-        .with_limit(limit)
-        .with_fixint_encoding() // As per https://github.com/servo/bincode/issues/333, these two options are needed
-        .allow_trailing_bytes() // to retain the behavior of bincode::deserialize with the new `options()` method
-        .deserialize_from(instruction_data)
-        .map_err(|_| InstructionError::InvalidInstructionData)
-}
-
-#[cfg(test)]
-pub mod tests {
-    use {super::*, solana_program::system_instruction::SystemInstruction};
-
-    #[test]
-    fn test_limited_deserialize_advance_nonce_account() {
-        let item = SystemInstruction::AdvanceNonceAccount;
-        let mut serialized = bincode::serialize(&item).unwrap();
-
-        assert_eq!(
-            serialized.len(),
-            4,
-            "`SanitizedMessage::get_durable_nonce()` may need a change"
-        );
-
-        assert_eq!(
-            limited_deserialize::<SystemInstruction>(&serialized, 4).as_ref(),
-            Ok(&item)
-        );
-        assert!(limited_deserialize::<SystemInstruction>(&serialized, 3).is_err());
-
-        serialized.push(0);
-        assert_eq!(
-            limited_deserialize::<SystemInstruction>(&serialized, 4).as_ref(),
-            Ok(&item)
-        );
-    }
-}
+pub use solana_bincode::limited_deserialize;