Submission Details
Severity:
Secret cannot be updated due to move_to
Root + Impact
set_secret unconditionally does move_to even if Vault already exists. A second call aborts with RESOURCE_ALREADY_EXISTS, preventing update of the secret.
Description
The owner should be able to rotate or update their secret. A second set_secret to the same address aborts and there is no function that specifically updates the secret in case the Vault already exists.
public entry fun set_secret(caller: &signer, secret: vector<u8>) {
let secret_vault = Vault { secret: string::utf8(secret) };
@> move_to(caller, secret_vault); // second call aborts
event::emit(SetNewSecret {});
}
Risk
Owner cannot update the secret and there is no function that updates a Vault if it already exists.
Proof of Concept
Consider running the following test :
fun test_set_secret_again(owner: &signer) {
use aptos_framework::account;
// Set up test environment
account::create_account_for_test(signer::address_of(owner));
let bytes = b"i'm a secret";
set_secret(owner, bytes);
set_secret(owner, bytes);
}
Recommended Mitigation
- public entry fun set_secret(caller: &signer, secret: vector<u8>) {
- let secret_vault = Vault { secret: string::utf8(secret) };
- move_to(caller, secret_vault);
- event::emit(SetNewSecret {});
- }
+ public entry fun set_secret(caller: &signer, secret: vector<u8>) acquires Vault {
+ let addr = signer::address_of(caller);
+ if (!exists<Vault>(addr)) {
+ move_to(caller, Vault { secret: string::utf8(secret) });
+ } else {
+ let v = borrow_global_mut<Vault>(addr);
+ v.secret = string::utf8(secret);
+ };
+ event::emit(SetNewSecret {});
+ }
Updates
Lead Judging Commences
bube 17 days ago
Submission Judgement Published Assigned finding tags:
The `secret` can not be updated
Rewards breakdown
nSLOC
30This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.