Miner: Export ChangeMultiaddrs

Author: arajasek

actors/miner/src/lib.rs

@@ -128,6 +128,7 @@ pub enum Method {
     ChangeWorkerAddressExported = frc42_dispatch::method_hash!("ChangeWorkerAddress"),
     ChangePeerIDExported = frc42_dispatch::method_hash!("ChangePeerID"),
     WithdrawBalanceExported = frc42_dispatch::method_hash!("WithdrawBalance"),
+    ChangeMultiaddrsExported = frc42_dispatch::method_hash!("ChangeMultiaddrs"),
     ChangeBenificiaryExported = frc42_dispatch::method_hash!("ChangeBeneficiary"),
     GetBeneficiaryExported = frc42_dispatch::method_hash!("GetBeneficiary"),
     GetOwnerExported = frc42_dispatch::method_hash!("GetOwner"),
@@ -5000,7 +5001,7 @@ impl ActorCode for Actor {
                 Self::confirm_sector_proofs_valid(rt, cbor::deserialize_params(params)?)?;
                 Ok(RawBytes::default())
             }
-            Some(Method::ChangeMultiaddrs) => {
+            Some(Method::ChangeMultiaddrs) | Some(Method::ChangeMultiaddrsExported) => {
                 Self::change_multiaddresses(rt, cbor::deserialize_params(params)?)?;
                 Ok(RawBytes::default())
             }

actors/miner/tests/miner_actor_test_peer_info.rs

@@ -1,6 +1,8 @@
 use fil_actors_runtime::test_utils::*;
 
-use fvm_ipld_encoding::BytesDe;
+use fil_actor_miner::{Actor, ChangeMultiaddrsParams, Method};
+use fvm_ipld_encoding::{BytesDe, RawBytes};
+use fvm_shared::error::ExitCode;
 
 mod util;
 
@@ -109,3 +111,41 @@ fn cant_set_large_multiaddrs() {
 
     h.check_state(&rt);
 }
+
+#[test]
+fn change_multiaddrs_restricted_correctly() {
+    let mut rt = MockRuntime::default();
+    let h = util::ActorHarness::new(0);
+
+    h.construct_and_verify(&mut rt);
+
+    let new_multiaddrs = vec![BytesDe(vec![1, 3, 3, 7])];
+
+    let params =
+        &RawBytes::serialize(ChangeMultiaddrsParams { new_multi_addrs: new_multiaddrs.clone() })
+            .unwrap();
+
+    rt.set_caller(make_identity_cid(b"1234"), h.worker);
+
+    // fail to call the unexported method
+
+    expect_abort_contains_message(
+        ExitCode::USR_FORBIDDEN,
+        "must be built-in",
+        rt.call::<Actor>(Method::ChangeMultiaddrs as u64, params),
+    );
+
+    // call the exported method
+
+    rt.expect_validate_caller_addr(h.caller_addrs());
+
+    rt.call::<Actor>(Method::ChangeMultiaddrsExported as u64, params).unwrap();
+    rt.verify();
+
+    let state = h.get_state(&rt);
+    let info = state.get_info(&rt.store).unwrap();
+
+    assert_eq!(new_multiaddrs, info.multi_address);
+
+    h.check_state(&rt);
+}