NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request from GHSA-93hq-5wgc-jc82

Browse Source 
Co-authored-by: Francisco <fg@frang.io>
This commit is contained in:
Hadrien Croubois
2023-04-13 20:47:51 +02:00
committed by GitHub
parent 3b117992e1
commit 8d633cb7d1
3 changed files with 22 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
.changeset/silent-pugs-scream.md Normal file
View File

@ -0,0 +1,5 @@
---
'openzeppelin-solidity': patch
---
`GovernorCompatibilityBravo`: Fix encoding of proposal data when signatures are missing.

4
contracts/governance/compatibility/GovernorCompatibilityBravo.sol
View File

@ -70,6 +70,7 @@ abstract contract GovernorCompatibilityBravo is IGovernorTimelock, IGovernorComp
bytes[] memory calldatas, bytes[] memory calldatas,
string memory description string memory description
) public virtual override returns (uint256) { ) public virtual override returns (uint256) {
require(signatures.length == calldatas.length, "GovernorBravo: invalid signatures length");
// Stores the full proposal and fallback to the public (possibly overridden) propose. The fallback is done // Stores the full proposal and fallback to the public (possibly overridden) propose. The fallback is done
// after the full proposal is stored, so the store operation included in the fallback will be skipped. Here we // after the full proposal is stored, so the store operation included in the fallback will be skipped. Here we
// call `propose` and not `super.propose` to make sure if a child contract override `propose`, whatever code // call `propose` and not `super.propose` to make sure if a child contract override `propose`, whatever code
@ -149,8 +150,7 @@ abstract contract GovernorCompatibilityBravo is IGovernorTimelock, IGovernorComp
bytes[] memory calldatas bytes[] memory calldatas
) private pure returns (bytes[] memory) { ) private pure returns (bytes[] memory) {
bytes[] memory fullcalldatas = new bytes[](calldatas.length); bytes[] memory fullcalldatas = new bytes[](calldatas.length);
for (uint256 i = 0; i < fullcalldatas.length; ++i) {
for (uint256 i = 0; i < signatures.length; ++i) {
fullcalldatas[i] = bytes(signatures[i]).length == 0 fullcalldatas[i] = bytes(signatures[i]).length == 0
? calldatas[i] ? calldatas[i]
: abi.encodePacked(bytes4(keccak256(bytes(signatures[i]))), calldatas[i]); : abi.encodePacked(bytes4(keccak256(bytes(signatures[i]))), calldatas[i]);

15
test/governance/compatibility/GovernorCompatibilityBravo.test.js
View File

@ -224,6 +224,21 @@ contract('GovernorCompatibilityBravo', function (accounts) {
}); });
}); });
it('with inconsistent array size for selector and arguments', async function () {
const target = this.receiver.address;
this.helper.setProposal(
{
targets: [target, target],
values: [0, 0],
signatures: ['mockFunction()'], // One signature
data: ['0x', this.receiver.contract.methods.mockFunctionWithArgs(17, 42).encodeABI()], // Two data entries
},
'<proposal description>',
);
await expectRevert(this.helper.propose({ from: proposer }), 'GovernorBravo: invalid signatures length');
});
describe('should revert', function () { describe('should revert', function () {
describe('on propose', function () { describe('on propose', function () {
it('if proposal does not meet proposalThreshold', async function () { it('if proposal does not meet proposalThreshold', async function () {