PointerAddress support for >u32::MAX

Fixes #364 in a way that requires the least changes.

Only users who are manually creating `Pointer` to make a new
`PointerAddress` from parts will be affected by this API change.

We could also alternatively change *ALL* references of `Slot` and `TransactionIndex`
to support `BigNum` instead of `u32`. This would be inline with the
alonzo.cddl binary spec, but would make use of the library more
annoying, especially from javascript/wasm, as well as requiring more code changes.

I don't see why you would have a transaction index greater than even a
single byte, let alone 2^32 - 1. But since that's how the binary spec
is, it's possible we'll run into on-chain data that do this in the future.

This isn't the only place where we run into issues like this  as there are
edge cases anywhere where `nint` is specified since the CBOR library we
support doesn't handle numbers between `i64::MIN` and `-u64::MAX`.

Author: rooooooooob

rust/src/address.rs

@@ -293,9 +293,9 @@ impl Address {
                                | (ptr.network & 0xF);
                 buf.push(header);
                 buf.extend(ptr.payment.to_raw_bytes());
-                buf.extend(variable_nat_encode(ptr.stake.slot.into()));
-                buf.extend(variable_nat_encode(ptr.stake.tx_index.into()));
-                buf.extend(variable_nat_encode(ptr.stake.cert_index.into()));
+                buf.extend(variable_nat_encode(from_bignum(&ptr.stake.slot)));
+                buf.extend(variable_nat_encode(from_bignum(&ptr.stake.tx_index)));
+                buf.extend(variable_nat_encode(from_bignum(&ptr.stake.cert_index)));
             },
             AddrType::Enterprise(enterprise) => {
                 let header: u8 = 0b0110_0000
@@ -393,9 +393,9 @@ impl Address {
                             network,
                             &payment_cred,
                             &Pointer::new(
-                                slot.try_into().map_err(|_| DeserializeError::new("Address.Pointer.slot", DeserializeFailure::CBOR(cbor_event::Error::ExpectedU32)))?,
-                                tx_index.try_into().map_err(|_| DeserializeError::new("Address.Pointer.tx_index", DeserializeFailure::CBOR(cbor_event::Error::ExpectedU32)))?,
-                                cert_index.try_into().map_err(|_| DeserializeError::new("Address.Pointer.cert_index", DeserializeFailure::CBOR(cbor_event::Error::ExpectedU32)))?)))
+                                &to_bignum(slot),
+                                &to_bignum(tx_index),
+                                &to_bignum(cert_index))))
                 },
                 // enterprise
                 0b0110 | 0b0111 => {
@@ -608,30 +608,30 @@ impl Deserialize for RewardAddress {
 #[wasm_bindgen]
 #[derive(Debug, Clone, Eq, Ord, PartialEq, PartialOrd)]
 pub struct Pointer {
-    slot: Slot,
-    tx_index: TransactionIndex,
-    cert_index: CertificateIndex,
+    slot: BigNum,
+    tx_index: BigNum,
+    cert_index: BigNum,
 }
 
 #[wasm_bindgen]
 impl Pointer {
-    pub fn new(slot: Slot, tx_index: TransactionIndex, cert_index: CertificateIndex) -> Self {
+    pub fn new(slot: &BigNum, tx_index: &BigNum, cert_index: &BigNum) -> Self {
         Self {
-            slot,
-            tx_index,
-            cert_index,
+            slot: slot.clone(),
+            tx_index: tx_index.clone(),
+            cert_index: cert_index.clone(),
         }
     }
 
-    pub fn slot(&self) -> Slot {
+    pub fn slot(&self) -> BigNum {
         self.slot.clone()
     }
 
-    pub fn tx_index(&self) -> TransactionIndex {
+    pub fn tx_index(&self) -> BigNum {
         self.tx_index.clone()
     }
 
-    pub fn cert_index(&self) -> CertificateIndex {
+    pub fn cert_index(&self) -> BigNum {
         self.cert_index.clone()
     }
 }
@@ -711,7 +711,7 @@ mod tests {
         let ptr = PointerAddress::new(
             25,
             &StakeCredential::from_keyhash(&Ed25519KeyHash::from([23; Ed25519KeyHash::BYTE_COUNT])),
-            &Pointer::new(2354556573, 127, 0));
+            &Pointer::new(&to_bignum(2354556573), &to_bignum(127), &to_bignum(0)));
         let addr = ptr.to_address();
         let addr2 = Address::from_bytes(addr.to_bytes()).unwrap();
         assert_eq!(addr.to_bytes(), addr2.to_bytes());
@@ -827,9 +827,9 @@ mod tests {
             .derive(0)
             .to_public();
         let spend_cred = StakeCredential::from_keyhash(&spend.to_raw_key().hash());
-        let addr_net_0 = PointerAddress::new(NetworkInfo::testnet().network_id(), &spend_cred, &Pointer::new(1, 2, 3)).to_address();
+        let addr_net_0 = PointerAddress::new(NetworkInfo::testnet().network_id(), &spend_cred, &Pointer::new(&to_bignum(1), &to_bignum(2), &to_bignum(3))).to_address();
         assert_eq!(addr_net_0.to_bech32(None).unwrap(), "addr_test1gz2fxv2umyhttkxyxp8x0dlpdt3k6cwng5pxj3jhsydzerspqgpsqe70et");
-        let addr_net_3 = PointerAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &Pointer::new(24157, 177, 42)).to_address();
+        let addr_net_3 = PointerAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &Pointer::new(&to_bignum(24157), &to_bignum(177), &to_bignum(42))).to_address();
         assert_eq!(addr_net_3.to_bech32(None).unwrap(), "addr1gx2fxv2umyhttkxyxp8x0dlpdt3k6cwng5pxj3jhsydzer5ph3wczvf2w8lunk");
     }
 
@@ -883,9 +883,9 @@ mod tests {
             .derive(0)
             .to_public();
         let spend_cred = StakeCredential::from_keyhash(&spend.to_raw_key().hash());
-        let addr_net_0 = PointerAddress::new(NetworkInfo::testnet().network_id(), &spend_cred, &Pointer::new(1, 2, 3)).to_address();
+        let addr_net_0 = PointerAddress::new(NetworkInfo::testnet().network_id(), &spend_cred, &Pointer::new(&to_bignum(1), &to_bignum(2), &to_bignum(3))).to_address();
         assert_eq!(addr_net_0.to_bech32(None).unwrap(), "addr_test1gpu5vlrf4xkxv2qpwngf6cjhtw542ayty80v8dyr49rf5egpqgpsdhdyc0");
-        let addr_net_3 = PointerAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &Pointer::new(24157, 177, 42)).to_address();
+        let addr_net_3 = PointerAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &Pointer::new(&to_bignum(24157), &to_bignum(177), &to_bignum(42))).to_address();
         assert_eq!(addr_net_3.to_bech32(None).unwrap(), "addr1g9u5vlrf4xkxv2qpwngf6cjhtw542ayty80v8dyr49rf5evph3wczvf2kd5vam");
     }
 
@@ -966,9 +966,9 @@ mod tests {
             .derive(0)
             .to_public();
         let spend_cred = StakeCredential::from_keyhash(&spend.to_raw_key().hash());
-        let addr_net_0 = PointerAddress::new(NetworkInfo::testnet().network_id(), &spend_cred, &Pointer::new(1, 2, 3)).to_address();
+        let addr_net_0 = PointerAddress::new(NetworkInfo::testnet().network_id(), &spend_cred, &Pointer::new(&to_bignum(1), &to_bignum(2), &to_bignum(3))).to_address();
         assert_eq!(addr_net_0.to_bech32(None).unwrap(), "addr_test1gqy6nhfyks7wdu3dudslys37v252w2nwhv0fw2nfawemmnqpqgps5mee0p");
-        let addr_net_3 = PointerAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &Pointer::new(24157, 177, 42)).to_address();
+        let addr_net_3 = PointerAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &Pointer::new(&to_bignum(24157), &to_bignum(177), &to_bignum(42))).to_address();
         assert_eq!(addr_net_3.to_bech32(None).unwrap(), "addr1gyy6nhfyks7wdu3dudslys37v252w2nwhv0fw2nfawemmnyph3wczvf2dqflgt");
     }
 
@@ -1041,4 +1041,13 @@ mod tests {
         let addr_net_3 = BaseAddress::new(NetworkInfo::mainnet().network_id(), &spend_cred, &stake_cred).to_address();
         assert_eq!(addr_net_3.to_bech32(None).unwrap(), "addr1x80de0mz3m9xmgtlmqqzu06s0uvfsczskdec8k7v4jhr7077mjlk9rk2dkshlkqq9cl4qlccnps9pvmns0duet9w8ulsylzv28");
     }
+
+    #[test]
+    fn pointer_address_big() {
+        let addr = Address::from_bech32("addr_test1grqe6lg9ay8wkcu5k5e38lne63c80h3nq6xxhqfmhewf645pllllllllllll7lupllllllllllll7lupllllllllllll7lc9wayvj").unwrap();
+        let ptr = PointerAddress::from_address(&addr).unwrap().stake;
+        assert_eq!(u64::MAX, from_bignum(&ptr.slot));
+        assert_eq!(u64::MAX, from_bignum(&ptr.tx_index));
+        assert_eq!(u64::MAX, from_bignum(&ptr.cert_index));
+    }
 }

rust/src/tx_builder.rs

@@ -1847,9 +1847,9 @@ mod tests {
                 NetworkInfo::testnet().network_id(),
                 &spend_cred,
                 &Pointer::new(
-                    0,
-                    0,
-                    0
+                    &to_bignum(0),
+                    &to_bignum(0),
+                    &to_bignum(0)
                 )
             ).to_address(),
             &TransactionInput::new(&genesis_id(), 0),