Skip to content

Commit

Permalink
fix: remove Buffer from transaction signature verification (#27868)
Browse files Browse the repository at this point in the history
  • Loading branch information
macalinao committed Nov 29, 2022
1 parent 7999f42 commit 3b2014d
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions web3.js/src/transaction/legacy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -699,14 +699,17 @@ export class Transaction {
/**
* @internal
*/
_verifySignatures(signData: Buffer, requireAllSignatures: boolean): boolean {
_verifySignatures(
signData: Uint8Array,
requireAllSignatures: boolean,
): boolean {
for (const {signature, publicKey} of this.signatures) {
if (signature === null) {
if (requireAllSignatures) {
return false;
}
} else {
if (!verify(signature, signData, publicKey.toBuffer())) {
if (!verify(signature, signData, publicKey.toBytes())) {
return false;
}
}
Expand Down

0 comments on commit 3b2014d

Please sign in to comment.