-
Notifications
You must be signed in to change notification settings - Fork 133
Fix pointer problem in UnmanagedVector #571
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kulikthebird
approved these changes
Nov 13, 2024
kulikthebird
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good change!
|
@Mergifyio backport release/1.5 release/2.0 release/2.1 |
✅ Backports have been created
|
This was referenced Nov 13, 2024
kulikthebird
pushed a commit
that referenced
this pull request
Nov 14, 2024
Fix pointer problem (backport #571)
kulikthebird
pushed a commit
that referenced
this pull request
Nov 14, 2024
Fix pointer problem (backport #571)
kulikthebird
pushed a commit
that referenced
this pull request
Nov 14, 2024
Fix pointer problem (backport #571)
MikkySnow
pushed a commit
to SigmaGmbH/swisstronik-chain
that referenced
this pull request
Feb 3, 2025
MikkySnow
pushed a commit
to SigmaGmbH/swisstronik-chain
that referenced
this pull request
Feb 3, 2025
MikkySnow
pushed a commit
to SigmaGmbH/swisstronik-chain
that referenced
this pull request
Feb 10, 2025
* feat: add migrations for v1.0.7 upgrade * feat: add script for simulation of chain upgrade with 1 validator * feat: add cli command to obtain verification holder by verification id * fix: fix issue with upgrade handler * feat: add hack to deploy Arachnid Proxy * refactor: more meaningful error * feat: add deployment of Arachnid Deterministic Deployer * feat: add is_revoked to QueryVerificationsDetailsResponse * feat: return only non-revoked verification details in x/compliance precompile * refactor: remove redundant line * refactor: return in precompile only verifications with active issuers * refactor: prohibit to convert credential with inactive issuer * feat: add ffi query to revoke verification * feat: add precompile handler to revoke verification * feat: update compliance proxy contract * test: add test for revocation by issuer * feat: provide caller address during credential revocation * feat: add check for caller during credential revocation * fix: fix issue with invalid non-revocation proof * feat: add VT_BIOMETRIC verification type * refactor: remove unused dependencies * feat: add convertCredential function to precompile interface * test: add test for converting credential * feat: add convert credential function signature * feat: add method to encode convert credential request * feat: add precompile handler to convert credential * feat: add cli command to get attached holder public key * chore: intermediate commit * feat: add convertCredential method to precompile * test: fix test for converting v1 verification to v2 * fix: applying fix for pointer problem from CosmWasm/wasmvm#571 * test: add test for non-revocation proof * refactor: remove unused consts and imports * refactor: add index variable * fix: fix build using `make build-docker-local` * ci: use actions/upload-artifact@v4 * fix: add checks in keeper for max sizes of verification data fields * test: add test for x/compliance keeper for case with huge original data * fix: add checks for verification details sizes in compliance_bridge precompile * feat: start refactoring self attestation to DCAP instead of deprecated EPID * feat: add DCAP-based self attestation for mainnet / testnet checks * chore: fix some clippy warnings * chore: run cargo clippy --fix
MikkySnow
pushed a commit
to SigmaGmbH/swisstronik-chain
that referenced
this pull request
Apr 7, 2025
* refactor: remove eip712 files * refactor: remove usages of eip712 * fix: fix build in simulation mode * fix: fix logIndex * Let's try * sudo let's try * refactor: cherry pick fix for log index (#127) * fix: fix build in simulation mode * feat: bump version to v1.0.4 * Fix build-deb to 1.0.4 * Sudo try again * Sudo try again#2 * fix: add upgrade handler for v1.0.4 * Try again with docker * Production = false, version = 1.0.4 * Fix dev key * Init enclave * Remove unused flag in setup app * Simulation testing with importing/exporting genesis * chore: small improvements * Make Full AppSimulation running * Test import/export genesis * Test simulation after importing genesis * Test app state determinism * Generate genesis vesting accounts * Unit test for generating vesting genesis account * Additional test cases for vesting module to check periods * Resolve balances in case of delegate/undelegate with additional integration test * Receive/send coins to/from mva * Optimize validation code lines * Additional test cases for vesting module to check periods * Resolve balances in case of delegate/undelegate with additional integration test * Receive/send coins to/from mva * Optimize validation code lines * Update test * Refactor structure of simapp test * Adjust test timeout * AS capital case * actually run it * Skip config test for CI * Don't run it * Revert "Don't run it" This reverts commit 0a5f501. * run it a little bit * Remove this branch * refactor: remove duplicated function * Mint 5M swtr to faucet addresses * Fix convert failure from eth pubkey to tendermint pubkey * Optimize msg delivery functions * Upgrade to v1.0.5 * Add integration test for different types of validators * Import order * Fix typo * Call markUserAsVerified() by eth_call * eth_estimateGas through unencrypted rpc * Update integration_validator_test.go * Separate tests for eth_call / eth_estimateGas * Update ComplianceBridge.js * Do only once * markUserAsVerified for random wallet * Structure compliance bridge tests * Remove redundant code * Refactor vesting ante handler * refactor: switch to github repo for evm * refactor: remove submodule * refactor: update submodule * refactor: bump rust version * refactor: change evm repo * refactor: upgrading evm backend * refactor: update go types * refactor: fix compilation issues in storage * refactor: fix compilation issues in updated backend * refactor: comment backend.rs content * refactor: fixing compilation issues in tx.rs * refactor: fix compilation issues in tx.rs * refactor: fix compilation issues in precompiles * refactor: fix compilation issues in precompiles * refactor: fix compilation issues in precompiles * refactor: fixing compilation issues * refactor: fixing compilation issues * refactor: fixing compilation issues * refactor: fixing compilation issues * refactor: all compilation issues * refactor: remove unused &mut self * chore: trying to make it work * refactor: provide gas price directly * refactor: return dummy error by default * refactor: fix compilation issues and sgx error INVALID_PARAMETER * refactor: apply some changes in backend * feat: add missing methods * fix: fixed issue with contract deployment * fix: restore some tests * refactor: remove redundant test * refactor: restore go tests * refactor: restore go and solidity tests * refactor: restore ERC20Unencrypted.js test * fix: restore solidity tests * fix: update gas estimation and error message in integration test for vesting * refactor: remove redundant console.log * refactor: remove redundant check for nonce * fix: restore compliance bridge test * fix: use similar behaviour as geth * refactor: bump solidity compiler version and openzeppelin * test: add test contract for inspecting block.* and msg.* fields * test: add test for unencrypted interaction with MsgInfo * test: add test for encrypted interaction with MsgInfo * test: add test for BlockInfo.sol * test: update test for BlockInfo.sol * fix: fix issue with panic on GetHashFn * fix: fix corner case for blockhash more than 256 blocks ago * fix: simplify check * test: add tests for opcodes * test: move to tokens folder * test: add test for WETH9 * test: add deployment of uniswap, creating pool and adding liquidity * test: add test for token swap * test: updating tests * fix: fix issue with returning an error * fix: fix compilation error * refactor: simplify code * refactor: fix error return * refactor: remove unused fn * test: fix naming + remove unused code * test: update solidity tests * test: comment redundant tests * test: update version to 1.0.6 * fix: add upgrade handler for v1.0.6 * remove toolchain from go.mod * remove unused insertAccount * add test for transient storage * restore go-sgxvm tests * refactor: move deoxys to crates folder * refactor: move static precompiles to separate crate * test: restore tests * refactor: remove commented code * refactor: reorganize code * feat: add config to enable unsafe eth endpoints + restore tests * test: update test env * test: add check if eth_sendTransaction is disabled by default * test: add check if eth_sign was disabled * refactor: remove unused eth_signTypedData endpoint * chore: add missing files to git * chore: fix missing params for nosgx build * Feat/zk sdi lfs (#139) * refactor: update gitignore * feat: add new tx to proto * feat: add revokeVerification msg handler * feat: add draft test for revokeVerification * feat: finish negative tests * feat: finish positive tests * feat: add mocked keeper functions * feat: add precompile for obtaining roots * feat: update querier * test: add tests with mocked values * test: add test for EVM request * test: fix issue with data len * feat: intermediate commit * feat: add functions to mark credential as issued / revoked in ZK-SDI * chore: update go.mod * chore: restore all tests * chore: add test for proof correctness * feat: add queries for zk-sdi * feat: fix issue with context * feat: add draft msgs and queries to set holder key * feat: add query if credential is included in ZK-SDI * chore: intermediate commit * feat: add methods to link holder and verification id * feat: restore holder address at genesis.go * test: restore tests * feat: add handler for credential conversion * test: restore test in genesis_test.go * test: add test for set / get holder public key * feat: add check for revoked credentials * test: add tests to attach public key * test: add test for addition to issuance tree * test: small fixes * test: add test for addVerification * test: add test for revoke verification * test: add test for credential conversion * feat: add AddVerificationDetailsV2 * refactor: update validation of public key * test: add test for AddVerificationDetailsV2 * test: fix tests * feat: add new compliance precompile methods * test: add test for precompile and update test contract * test: fix function names * refactor: chore * feat: bump version * feat: migrate debug_precompile.sh into local-node script * fix: fix crash in precompile * fix: fix duplicates in SMT * chore: add verification id to logs * add draft contract * chore: add missing files * feat: add draft frontend contract * feat: add endpoint for issuance proof / revocation proof * feat: add endpoint for getting credential hash * chore: intermediate changes * fix: fix issue with decompression of public key * fix: fix issue with SMT proof * fix: fix issue with verification details * tests: restore tests * chore: update .gitattributes * chore: add .zkey and .wasm * fix: construct correct SMT proofs * test: construct proof input * fix: fix issue with invalid credential hash * chore: remove commented code * test: constructed and verified sdi proof in test * test: verify sdi proof on-chain * test: restoring tests * test: restore tests * test: cleanup * test: update SDI test with MiMC * refactor: migrating on MiMC7 * fix: fix issue with SMT * chore: remove debug logs * feat: add export to genesis.json * feat: add CLI commands to set BJJ pubkey and convert credential * fix: fix issue with slice modification * feat: restore data with linked pubkeys and holders * refactor: disable functionality with handleVerifyIssuerProposal * refactor: remove redundant fields from genesis.json * refactor: comment tests for disabled functionality * feat: restore trees from genesis.json * refactor: update handler for revoke verification msg * refactor: add cli command to revoke verification * refactor: disable x/compliance proposal handler * tests: restore tests * refactor: replace usages of deprecated sdkerrors.wrap * chore: update README.md * v1.0.7 Upgrade (#140) * feat: add migrations for v1.0.7 upgrade * feat: add script for simulation of chain upgrade with 1 validator * feat: add cli command to obtain verification holder by verification id * fix: fix issue with upgrade handler * feat: add hack to deploy Arachnid Proxy * refactor: more meaningful error * feat: add deployment of Arachnid Deterministic Deployer * feat: add is_revoked to QueryVerificationsDetailsResponse * feat: return only non-revoked verification details in x/compliance precompile * refactor: remove redundant line * refactor: return in precompile only verifications with active issuers * refactor: prohibit to convert credential with inactive issuer * feat: add ffi query to revoke verification * feat: add precompile handler to revoke verification * feat: update compliance proxy contract * test: add test for revocation by issuer * feat: provide caller address during credential revocation * feat: add check for caller during credential revocation * fix: fix issue with invalid non-revocation proof * feat: add VT_BIOMETRIC verification type * refactor: remove unused dependencies * feat: add convertCredential function to precompile interface * test: add test for converting credential * feat: add convert credential function signature * feat: add method to encode convert credential request * feat: add precompile handler to convert credential * feat: add cli command to get attached holder public key * chore: intermediate commit * feat: add convertCredential method to precompile * test: fix test for converting v1 verification to v2 * fix: applying fix for pointer problem from CosmWasm/wasmvm#571 * test: add test for non-revocation proof * refactor: remove unused consts and imports * refactor: add index variable * fix: fix build using `make build-docker-local` * ci: use actions/upload-artifact@v4 * fix: add checks in keeper for max sizes of verification data fields * test: add test for x/compliance keeper for case with huge original data * fix: add checks for verification details sizes in compliance_bridge precompile * feat: start refactoring self attestation to DCAP instead of deprecated EPID * feat: add DCAP-based self attestation for mainnet / testnet checks * chore: fix some clippy warnings * chore: run cargo clippy --fix * Fix build deb pipeline (#141) * fix: update build-deb.yml pipeline * chore: remove branch rule * Bump power reduction (#142) * fix: prevent uint64 overflow in AddTransientGasWanted * fix: add check for zero value in validateElasticityMultiplier * fix: add compile_error! if both simulation_mode and hardware_mode features selected * chore: fix docker warning for different capitalization of FROM / AS * refactor: replace default power reduction in app.go * fix: restore RandomGenesisAccounts for vesting simulation * refactor: use our fork of cosmos-sdk * fix: restore feemarket tests * fix: fix issue in simulation of vesting * fix: do not update evmDenom during UpdateParams in x/evm * refactor: update gas prices for Ed25519Verify (2000), Curve25519Add (150), Curve25519Mul (6000) * refactor: move Ed25519Verifier contract to precompiles folder * refactor: set correct contract name * fix: add check for 0 gas in fee_checker.go * refactor: update blake2f precompile to meet EIP-152 gas cost * refactor: update bn128 gas costs according to EIP-1108 * add explicit check of quote size and collateral expiration status (#143) * feat: Check mainnet hardware (#144) * add MAINNET_MODE and CHECKER_MODE env args * feat: add checker_mode build feature for wrapper * feat: add build_checker make command * fix: fix issue with gas calculaton for bn128 pairing * Add migration for power reduction (#146) * feat: add dummy handler for v1.0.8 upgrade * refactor: bump version to v1.0.8 * docs: specify commit in license * refactor: bump version in scripts for upgrade testing * refactor: change upgrade height to 70 * fix: allow high s-values in ECRecover precompile * test: add tests for EC Recover corner-cases * test: move ComplianceBridge.sol to precompiles folder * chore: recompile proto * fix: fix gas calculation for bn128 pairing * chore: fix typo * chore: add debug script to reproduce issue with exceeding voting power * chore: prepare script for simulation with near-overflowing voting power * feat: add draft of upgrade handler * chore: compile proto * fix: fix possible overflow in AddTransientGasWanted * fix: support zero-cliff vesting * feat: add param `onlyWithExistingIssuer` to AddressDetails request * test: add test for GetAddressDetails and GetFullAddressDetails * refactor: move Export keeper methods to export.go * refactor: move methods to iterate through storage to iterators.go * refactor: move Export methods and iterators to separate files * feat: add AllVerificationDetailsByAddress query * test: add basic test for QueryAllVerificationDetails * test: restore tests * feat: update handler for convert credential * refactor: remove redundant todo * refactor: fix typo * feat: add helper type for unsigned transaction * chore: mark possible issue * feat: add function to recover tx sender * refactor: cleanup * refactor: add messages for estimategas * refactor: update compiled protobuf * fix: fix compilation issues * feat: broadcast call / tx signature into enclave * feat: provide max_fee_per_gas and max_priority_fee to restore tx hash * feat: recover tx sender in enclave * feat: add check if calculated sender is different than provided * fix: fix issue with conversion to u256 * fix: add normalization of V * feat: provide tx type to enclave * feat: add decoding of tx type * refactor: fix typos * feat: add SGXVMEstimateGasRequest request * feat: add handler for SGXVMEstimateGasRequest request * feat: update protobuf * feat: add EstimateGas to librustgo * feat: add wrapper for EstimateGas to librustgo * feat: update estimate gas function * refactor: try to recover sender only if non-zero address .from is provided * refactor: restore ECRecover test * refactor: make signed calls working * refactor: restore go tests * refactor: remove commented code * Update x/compliance (#147) * fix: add fix for biometric type * feat: add check for potential overflow * test: add check for successful transaction * Fix/restore pipelines (#148) * refactor: update branches in pipelines * fix: restore nosgx build * refactor: remove current branch from on-push rule * Add mainnet MRSIGNER (#150) * fix: restore make commands to support already signed enclave * feat: add mainnet MRSIGNER * feat: disable EPID support * fix: restore test for AddVerificationDetails * fix: restore CLI build * Remove more EPID stuff * Run tests on every push * Run tests on every push #2 * fix: restore build --------- Co-authored-by: Denis <[email protected]> * Refactor: remove unused EPID code (#151) * refactor: comment unused dependencies in sgxvm * refactor: remove unused code and imports * refactor: remove commented code * refactor: remove & comment redundant code * fix: restore tests * feat: allow to add 0 epoch using attestationServer binary * feat: apply changes to build separate enclave * fix: restore build * refactor: remove commented code * refactor: remove unused function * refactor: remove unused dependencies * refactor: remove unused braces * refactor: update go.mod * refactor: enable hardware_mode if attestation_server feature was selected * Feat: update compliance queries (#152) * refactor: update protobuf * feat: return MergedVerificationDetails * refactor: prettify query cli for compliance * refactor: accept both bech32 and hex addresses + make parameters more meaningful * Add scripts for testing (#153) * chore: add script for validator creation * chore: add script to create 1 validator and 1 account with big balance * chore: add script to setup chain with vesting accounts * chore: add duplicated vesting account for testing * refactor: return more meaningfull error * refactor: update genesis-vesting script * chore: add command to delegate from vesting account * chore: add script to check if it's possible to delegate from vesting account created from CLI * chore: fund vesting account with gas coins * fix: created validator with 310mln SWTR staked * fix: increase delegation amount to make validator bonded * fix: restore tests * fix: fix inflation params to appropriate values * Feat/separate build process for enclave (#149) * feat: add command to build enclave only * feat: add command to build swisstronikd with ready enclave * refactor: remove duplicated command from Makefile * Feat/mainnet changes (#154) * refactor: change binary and lib version to v1.0.0 * refactor: remove testnet upgrades * refactor: set mainnet chain id * refactor: move docker-compose files to separate folder * refactor: remove unused ignite config * refactor: remove test upgrade script for testnet * refactor: change chain id to 1848 * refactor: remove unused scripts * fix: update V value in test for 1848 chain id * fix: restore deb build pipeline (#156) * fix: restore deb build pipeline * fix: remove branch from on push rule --------- Co-authored-by: Denis <[email protected]> Co-authored-by: Hungry Warrior <[email protected]> Co-authored-by: deep-quality-dev <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes the problem reported in CosmWasm/wasmd#2011