⚡️ Use an addr instead of str for map key

- avoid any issues with upper/lower case that can cause a bypass of the blocked address list

contracts/entry-point/src/contract.rs

@@ -24,7 +24,7 @@ pub fn instantiate(
     let mut response: Response = Response::new().add_attribute("action", "instantiate");
 
     // Insert the entry point contract address into the blocked contract addresses map
-    BLOCKED_CONTRACT_ADDRESSES.save(deps.storage, env.contract.address.as_str(), &())?;
+    BLOCKED_CONTRACT_ADDRESSES.save(deps.storage, &env.contract.address, &())?;
 
     // Iterate through the swap venues provided and create a map of venue names to swap adapter contract addresses
     for swap_venue in msg.swap_venues.iter() {
@@ -46,7 +46,7 @@ pub fn instantiate(
         )?;
 
         // Insert the swap contract address into the blocked contract addresses map
-        BLOCKED_CONTRACT_ADDRESSES.save(deps.storage, &swap_venue.adapter_contract_address, &())?;
+        BLOCKED_CONTRACT_ADDRESSES.save(deps.storage, &checked_swap_contract_address, &())?;
 
         // Add the swap venue and contract address to the response
         response = response
@@ -63,7 +63,7 @@ pub fn instantiate(
     IBC_TRANSFER_CONTRACT_ADDRESS.save(deps.storage, &checked_ibc_transfer_contract_address)?;
 
     // Insert the ibc transfer adapter contract address into the blocked contract addresses map
-    BLOCKED_CONTRACT_ADDRESSES.save(deps.storage, &msg.ibc_transfer_contract_address, &())?;
+    BLOCKED_CONTRACT_ADDRESSES.save(deps.storage, &checked_ibc_transfer_contract_address, &())?;
 
     // Add the ibc transfer adapter contract address to the response
     response = response

contracts/entry-point/src/execute.rs

@@ -326,10 +326,10 @@ fn verify_and_create_contract_call_msg(
     transfer_out_coin: Coin,
 ) -> ContractResult<WasmMsg> {
     // Verify the contract address is valid, error if invalid
-    deps.api.addr_validate(&contract_address)?;
+    let checked_contract_address = deps.api.addr_validate(&contract_address)?;
 
     // Error if the contract address is in the blocked contract addresses map
-    if BLOCKED_CONTRACT_ADDRESSES.has(deps.storage, &contract_address) {
+    if BLOCKED_CONTRACT_ADDRESSES.has(deps.storage, &checked_contract_address) {
         return Err(ContractError::ContractCallAddressBlocked);
     }
 

contracts/entry-point/src/state.rs

@@ -1,6 +1,6 @@
 use cosmwasm_std::Addr;
 use cw_storage_plus::{Item, Map};
 
-pub const BLOCKED_CONTRACT_ADDRESSES: Map<&str, ()> = Map::new("blocked_contract_addresses");
+pub const BLOCKED_CONTRACT_ADDRESSES: Map<&Addr, ()> = Map::new("blocked_contract_addresses");
 pub const SWAP_VENUE_MAP: Map<&str, Addr> = Map::new("swap_venue_map");
 pub const IBC_TRANSFER_CONTRACT_ADDRESS: Item<Addr> = Item::new("ibc_transfer_contract_address");

contracts/entry-point/tests/test_execute_post_swap_action.rs

@@ -553,7 +553,7 @@ fn test_execute_post_swap_action(params: Params) {
 
     // Store the entry point contract address in the blocked contract addresses map
     BLOCKED_CONTRACT_ADDRESSES
-        .save(deps.as_mut().storage, "entry_point", &())
+        .save(deps.as_mut().storage, &Addr::unchecked("entry_point"), &())
         .unwrap();
 
     // Call execute_post_swap_action with the given test parameters