Revert "Support 20 bytes address"

This reverts commit 69e4e59.

Author: lxfind

Cargo.lock

@@ -17,7 +17,6 @@ petgraph = "0.5.1"
 walkdir = "2.3.1"
 tempfile = "3.2.0"
 once_cell = "1.7.2"
-num-bigint = "0.4.0"
 
 move-binary-format = { path = "../move-binary-format" }
 move-core-types = { path = "../move-core/types" }

language/move-compiler/Cargo.toml

@@ -8,7 +8,6 @@ use crate::{
 use move_core_types::account_address::AccountAddress;
 use move_ir_types::location::*;
 use move_symbol_pool::Symbol;
-use num_bigint::BigUint;
 use petgraph::{algo::astar as petgraph_astar, graphmap::DiGraphMap};
 use std::{
     collections::BTreeMap,
@@ -62,25 +61,6 @@ pub fn parse_u128(s: &str) -> Result<(u128, NumberFormat), ParseIntError> {
     Ok((u128::from_str_radix(txt, base as u32)?, base))
 }
 
-// Parse an address from a decimal or hex encoding
-pub fn parse_address(s: &str) -> Option<([u8; AccountAddress::LENGTH], NumberFormat)> {
-    let (txt, base) = determine_num_text_and_base(s);
-    let parsed = BigUint::parse_bytes(
-        txt.as_bytes(),
-        match base {
-            NumberFormat::Hex => 16,
-            NumberFormat::Decimal => 10,
-        },
-    )?;
-    let bytes = parsed.to_bytes_be();
-    if bytes.len() > AccountAddress::LENGTH {
-        return None;
-    }
-    let mut result = [0u8; AccountAddress::LENGTH];
-    result[(AccountAddress::LENGTH - bytes.len())..].clone_from_slice(&bytes);
-    Some((result, base))
-}
-
 //**************************************************************************************************
 // Address
 //**************************************************************************************************
@@ -98,7 +78,9 @@ pub struct NumericalAddress {
 impl NumericalAddress {
     // bytes used for errors when an address is not known but is needed
     pub const DEFAULT_ERROR_ADDRESS: Self = NumericalAddress {
-        bytes: AccountAddress::ONE,
+        bytes: AccountAddress::new([
+            0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 1u8,
+        ]),
         format: NumberFormat::Hex,
     };
 
@@ -118,22 +100,22 @@ impl NumericalAddress {
     }
 
     pub fn parse_str(s: &str) -> Result<NumericalAddress, String> {
-        match parse_address(s) {
-            Some((n, format)) => Ok(NumericalAddress {
-                bytes: AccountAddress::new(n),
-                format,
-            }),
-            None =>
-            // TODO the kind of error is in an unstable nightly API
-            // But currently the only way this should fail is if the number is too long
-            {
-                Err(format!(
+        let (n, format) = match parse_u128(s) {
+            Ok(res) => res,
+            Err(_) => {
+                // TODO the kind of error is in an unstable nightly API
+                // But currently the only way this should fail is if the number is too long
+                return Err(
                     "Invalid address literal. The numeric value is too large. The maximum size is \
-                     {} bytes",
-                    AccountAddress::LENGTH
-                ))
+                     16 bytes"
+                        .to_owned(),
+                );
             }
-        }
+        };
+        Ok(NumericalAddress {
+            bytes: AccountAddress::new(n.to_be_bytes()),
+            format,
+        })
     }
 }
 
@@ -147,7 +129,7 @@ impl fmt::Display for NumericalAddress {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         match self.format {
             NumberFormat::Decimal => {
-                let n = BigUint::from_bytes_be(self.bytes.as_ref());
+                let n = u128::from_be_bytes(self.bytes.into_bytes());
                 write!(f, "{}", n)
             }
             NumberFormat::Hex => write!(f, "{:#X}", self),

language/move-compiler/src/shared/mod.rs

@@ -30,6 +30,5 @@ regex = "1.4.3"
 serde_json = "1.0.64"
 
 [features]
-address20 = []
 default = []
 fuzzing = ["proptest", "proptest-derive"]

language/move-core/types/Cargo.toml

@@ -17,21 +17,11 @@ impl AccountAddress {
     }
 
     /// The number of bytes in an address.
-    /// Default to 16 bytes, can be set to 20 bytes with address20 feature.
-    pub const LENGTH: usize = if cfg!(feature = "address20") { 20 } else { 16 };
+    pub const LENGTH: usize = 16;
 
     /// Hex address: 0x0
     pub const ZERO: Self = Self([0u8; Self::LENGTH]);
 
-    /// Hex address: 0x1
-    pub const ONE: Self = Self::get_hex_address_one();
-
-    const fn get_hex_address_one() -> Self {
-        let mut addr = [0u8; AccountAddress::LENGTH];
-        addr[AccountAddress::LENGTH - 1] = 1u8;
-        Self(addr)
-    }
-
     pub fn random() -> Self {
         let mut rng = OsRng;
         let buf: [u8; Self::LENGTH] = rng.gen();

language/move-core/types/src/account_address.rs

@@ -13,8 +13,9 @@ use std::fmt::{Display, Formatter};
 pub const CODE_TAG: u8 = 0;
 pub const RESOURCE_TAG: u8 = 1;
 
-/// Hex address: 0x1
-pub const CORE_CODE_ADDRESS: AccountAddress = AccountAddress::ONE;
+pub const CORE_CODE_ADDRESS: AccountAddress = AccountAddress::new([
+    0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 0u8, 1u8,
+]);
 
 #[derive(Serialize, Deserialize, Debug, PartialEq, Hash, Eq, Clone, PartialOrd, Ord)]
 pub enum TypeTag {

language/move-core/types/src/language_storage.rs

@@ -41,7 +41,8 @@ pub type EvalResult<T> = ::std::result::Result<T, BigInt>;
 // Constants
 //**************************************************************************************************
 
-const DIEM_CORE_ADDR: AccountAddress = AccountAddress::ONE;
+const DIEM_CORE_ADDR: AccountAddress =
+    AccountAddress::new([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1]);
 
 //**************************************************************************************************
 // Evaluation context

language/move-prover/interpreter/src/concrete/evaluator.rs

@@ -13,7 +13,6 @@ use bytecode_interpreter_crypto::{
 use move_binary_format::errors::Location;
 use move_core_types::{
     account_address::AccountAddress,
-    language_storage::CORE_CODE_ADDRESS,
     vm_status::{sub_status, StatusCode},
 };
 use move_model::{
@@ -54,7 +53,8 @@ pub type ExecResult<T> = ::std::result::Result<T, AbortInfo>;
 // Constants
 //**************************************************************************************************
 
-const DIEM_CORE_ADDR: AccountAddress = CORE_CODE_ADDRESS;
+const DIEM_CORE_ADDR: AccountAddress =
+    AccountAddress::new([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1]);
 
 // TODO(mengxu): these constants are defined in values_impl.rs which are currently not exposed.
 const INDEX_OUT_OF_BOUNDS: u64 = sub_status::NFE_VECTOR_ERROR_BASE + 1;

language/move-prover/interpreter/src/concrete/player.rs

@@ -12,13 +12,6 @@ use std::collections::VecDeque;
 
 use move_core_types::account_address::AccountAddress;
 
-fn to_le_bytes(i: u64) -> [u8; AccountAddress::LENGTH] {
-    let bytes = i.to_le_bytes();
-    let mut result = [0u8; AccountAddress::LENGTH];
-    result[..bytes.len()].clone_from_slice(bytes.as_ref());
-    result
-}
-
 pub fn native_create_signers_for_testing(
     _context: &mut NativeContext,
     ty_args: Vec<Type>,
@@ -29,7 +22,7 @@ pub fn native_create_signers_for_testing(
 
     let num_signers = pop_arg!(args, u64);
     let signers = Value::vector_for_testing_only(
-        (0..num_signers).map(|i| Value::signer(AccountAddress::new(to_le_bytes(i)))),
+        (0..num_signers).map(|i| Value::signer(AccountAddress::new((i as u128).to_le_bytes()))),
     );
 
     Ok(NativeResult::ok(ONE_GAS_UNIT, smallvec![signers]))