Skip to content

Commit 5666931

Browse files
fix(DK): hash justification
1 parent 8af1ba0 commit 5666931

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

contracts/src/arbitration/dispute-kits/DisputeKitGatedShutter.sol

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,14 +108,14 @@ contract DisputeKitGatedShutter is DisputeKitClassicBase {
108108
/// @param _coreDisputeID The ID of the dispute in Kleros Core.
109109
/// @param _voteIDs The IDs of the votes.
110110
/// @param _recoveryCommit The commitment hash without the justification.
111-
/// @param _justification Justification of the choice.
111+
/// @param _justificationHash Hash of the justification of the choice.
112112
/// @param _identity The Shutter identity used for encryption.
113113
/// @param _encryptedVote The Shutter encrypted vote.
114114
function castCommitShutter(
115115
uint256 _coreDisputeID,
116116
uint256[] calldata _voteIDs,
117117
bytes32 _recoveryCommit,
118-
string memory _justification,
118+
bytes32 _justificationHash,
119119
bytes32 _identity,
120120
bytes calldata _encryptedVote
121121
) external notJumped(_coreDisputeID) {
@@ -128,8 +128,7 @@ contract DisputeKitGatedShutter is DisputeKitClassicBase {
128128
recoveryCommitments[localDisputeID][localRoundID][_voteIDs[i]] = _recoveryCommit;
129129
}
130130
// Construct Shutter commit out of recovery commit + justification.
131-
bytes32 justificationHash = keccak256(bytes(_justification));
132-
bytes32 commit = keccak256(abi.encode(_recoveryCommit, justificationHash));
131+
bytes32 commit = keccak256(abi.encode(_recoveryCommit, _justificationHash));
133132
// `_castCommit()` ensures that the caller owns the vote
134133
_castCommit(_coreDisputeID, _voteIDs, commit);
135134
emit CommitCastShutter(_coreDisputeID, msg.sender, commit, _recoveryCommit, _identity, _encryptedVote);

0 commit comments

Comments
 (0)