fix: comments fix 1

nssa/src/encoding/privacy_preserving_transaction.rs

@@ -5,7 +5,7 @@ use crate::{
 
 impl Message {
     pub fn to_bytes(&self) -> Vec<u8> {
-        borsh::to_vec(&self).unwrap()
+        borsh::to_vec(&self).expect("Autoderived borsh serialization failure")
     }
 
     pub fn from_bytes(bytes: &[u8]) -> Result<Self, NssaError> {
@@ -15,7 +15,7 @@ impl Message {
 
 impl PrivacyPreservingTransaction {
     pub fn to_bytes(&self) -> Vec<u8> {
-        borsh::to_vec(&self).unwrap()
+        borsh::to_vec(&self).expect("Autoderived borsh serialization failure")
     }
 
     pub fn from_bytes(bytes: &[u8]) -> Result<Self, NssaError> {

nssa/src/encoding/program_deployment_transaction.rs

@@ -2,7 +2,7 @@ use crate::{ProgramDeploymentTransaction, error::NssaError};
 
 impl ProgramDeploymentTransaction {
     pub fn to_bytes(&self) -> Vec<u8> {
-        borsh::to_vec(&self).unwrap()
+        borsh::to_vec(&self).expect("Autoderived borsh serialization failure")
     }
 
     pub fn from_bytes(bytes: &[u8]) -> Result<Self, NssaError> {

nssa/src/encoding/public_transaction.rs

@@ -2,13 +2,13 @@ use crate::{PublicTransaction, error::NssaError, public_transaction::Message};
 
 impl Message {
     pub(crate) fn to_bytes(&self) -> Vec<u8> {
-        borsh::to_vec(&self).unwrap()
+        borsh::to_vec(&self).expect("Autoderived borsh serialization failure")
     }
 }
 
 impl PublicTransaction {
     pub fn to_bytes(&self) -> Vec<u8> {
-        borsh::to_vec(&self).unwrap()
+        borsh::to_vec(&self).expect("Autoderived borsh serialization failure")
     }
 
     pub fn from_bytes(bytes: &[u8]) -> Result<Self, NssaError> {

nssa/src/signature/public_key.rs

@@ -5,9 +5,23 @@ use crate::{PrivateKey, error::NssaError};
 
 use sha2::{Digest, Sha256};
 
-#[derive(Debug, Clone, PartialEq, Eq, BorshSerialize, BorshDeserialize)]
+#[derive(Debug, Clone, PartialEq, Eq, BorshSerialize)]
 pub struct PublicKey([u8; 32]);
 
+impl BorshDeserialize for PublicKey {
+    fn deserialize_reader<R: std::io::Read>(reader: &mut R) -> std::io::Result<Self> {
+        let mut buf = [0u8; 32];
+        reader.read_exact(&mut buf)?;
+
+        Self::try_new(buf).map_err(|_| {
+            std::io::Error::new(
+                std::io::ErrorKind::InvalidData,
+                "Invalid public key: not a valid point",
+            )
+        })
+    }
+}
+
 impl PublicKey {
     pub fn new_from_private_key(key: &PrivateKey) -> Self {
         let value = {
@@ -94,4 +108,14 @@ mod test {
             );
         }
     }
+
+    #[test]
+    fn test_correct_ser_deser_roundtrip() {
+        let pub_key = PublicKey::try_new([42; 32]).unwrap();
+
+        let pub_key_borsh_ser = borsh::to_vec(&pub_key).unwrap();
+        let pub_key_new: PublicKey = borsh::from_slice(&pub_key_borsh_ser).unwrap();
+
+        assert_eq!(pub_key, pub_key_new);
+    }
 }