Skip to content

Commit bd21746

Browse files
authored
bugfix: update internal owner address in transferred share (#6139)
1 parent 59b62fa commit bd21746

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

contracts/coconut-dkg/src/dealers/transactions.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,9 @@ pub fn try_transfer_ownership(
118118
EPOCH_DEALERS_MAP.remove(deps.storage, (epoch_id, &info.sender));
119119
EPOCH_DEALERS_MAP.save(deps.storage, (epoch_id, &transfer_to), &details)?;
120120
}
121-
if let Some(vk_share) = vk_shares().may_load(deps.storage, (&info.sender, epoch_id))? {
121+
if let Some(mut vk_share) = vk_shares().may_load(deps.storage, (&info.sender, epoch_id))? {
122122
vk_shares().remove(deps.storage, (&info.sender, epoch_id))?;
123+
vk_share.owner = transfer_to.clone();
123124
vk_shares().save(deps.storage, (&transfer_to, epoch_id), &vk_share)?;
124125
}
125126
}
@@ -307,7 +308,9 @@ mod tests_with_mock {
307308
// the underlying info hasn't changed
308309
assert_eq!(old_index, new_index);
309310
assert_eq!(old_details, new_details);
310-
assert_eq!(old_share, new_share);
311+
assert_ne!(old_share, new_share);
312+
assert_eq!(old_share.owner, group_member);
313+
assert_eq!(new_share.owner, new_group_member);
311314

312315
assert_eq!(
313316
OWNERSHIP_TRANSFER_LOG.load(

0 commit comments

Comments
 (0)